Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[톰캣 구현하기 - 3, 4단계] 베베(최원용) 미션 제출합니다. #417

Merged
merged 25 commits into from
Sep 14, 2023

Conversation

wonyongChoi05
Copy link
Member

안녕하세여 민트. 저번 코드리뷰에서 많이 배웠습니다! 이번에도 잘 부탁드립니다 ㅎㅎㅎ

Copy link
Member

@yujamint yujamint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 베베! 민트입니다. 이번에도 리뷰가 좀 늦었죠,,🥲 제 미션 진행 속도가 느려서 리뷰까지 늦게 하게 되네요,,,

1,2단계와 마찬가지로 3,4단계도 잘 진행하셨네요. 고생하셨습니다~
역시나 커밋 단위를 작게, 잘 해주신 덕분에 보기 편했습니다 ㅎㅎ
코멘트 남긴 부분 확인 부탁드립니다!

@@ -24,6 +36,12 @@ public Connector() {

public Connector(final int port, final int acceptCount) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰라기보단, 저도 궁금했던 부분이라 베베에게도 여쭤보고 싶은 것이 있습니다!

// maxThreads를 추가했다.
public Connector(final Container container, final int port, final int acceptCount, final int maxThreads) {
    // 생성자에서 스레드 풀 생성
}

4단계 미션 요구사항을 보면, Connector 생성자를 다음과 같은 시그니처로 정의하고 있습니다.
여기서 Container 객체는 과연 어떤 역할을 하는 객체일까요?
스레드 관련하여 지식이 거의 없기 때문에 감이 안 잡히네요 ㅎㅎ.. 혹시 베베는 짐작이 가시나요?!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Connector 클래스의 생성자에서 파라미터로 받는 Container는 아마 웹 서버에서 사용되는 컨테이너를 나타내는 객체일 것입니다.(서블릿 컨테이너일듯?)

Connector 클래스의 생성자에서 받는 Container 객체는 뭔가 컨테이너와 관련된 설정이나 동작을 구성하고 초기화하는 데 사용될 것 같네요. 구체적인 내용은 코드의 나머지 부분이나 설명을 확인해봐야 알 것 같은데 별도로 제공되지는 않나보군요 ㅠ

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아하 .. 이 부분에 대해서는 따로 공부가 필요한 것 같네요 ㅋㅋㅋ 저도 잘 몰라서 사용하지 않았습니다.. 의견 알려주셔서 감사합니다~!

Copy link
Member

@yujamint yujamint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 반영하느라 고생하셨어요 베베
제대로 동작하고 있지 않은 부분에 대해서 코멘트 남겼습니다. 해당 부분만 수정하면 이번 미션 끝내도 좋을 거 같네요. 마지막 리뷰 반영도 파이팅입니다~

@@ -24,6 +36,12 @@ public Connector() {

public Connector(final int port, final int acceptCount) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아하 .. 이 부분에 대해서는 따로 공부가 필요한 것 같네요 ㅋㅋㅋ 저도 잘 몰라서 사용하지 않았습니다.. 의견 알려주셔서 감사합니다~!

@sonarcloud
Copy link

sonarcloud bot commented Sep 12, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 13 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Copy link
Member

@yujamint yujamint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

베베 리뷰 반영하느라 고생하셨습니다! 머지하도록 하겠습니다~
톰캣 미션 고생하셨고 MVC 미션 파이팅하세요!

@yujamint yujamint merged commit 3bae892 into woowacourse:wonyongchoi05 Sep 14, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants