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

[4단계 - Tomcat 구현하기] 마크(김승현) 미션 제출합니다. #751

Merged
merged 44 commits into from
Sep 19, 2024

Conversation

seunghye218
Copy link

@seunghye218 seunghye218 commented Sep 13, 2024

안녕하세요 미아~~ 급하게 제출했어서 PR 본문 작성을 놓쳤었네요...! 학습 테스트를 통해 자바에서 스레드를 어떻게 다루는지 학습하여 동시성을 확장하였어요. 마지막 단계 리뷰 잘 부탁드려요~~

geoje and others added 30 commits September 5, 2024 11:11
- MimeType 열거형 추가
- 정적 URI를 못찾을 시 404 반환
- getStaticResource(HttpRequest request) -> getStaticResourceResponse(String requestPath)
- CacheControlInterceptor 추가
- ShallowEtagHeaderFilter 사용
- 캐시 max-age 1년 설정
- ETag 적용
- url에 버전 적용
- 요청 본문 파싱 로직 수정
- HttpCookie 객체 구현 및 요청, 응답 객체에 추가
- 컨트롤러 인터페이스 추가
- 정적 자원 불러오는 로직 객체 분리
- 요청 경로애 따른 컨트롤러 맵핑
@seunghye218 seunghye218 requested a review from jongmee September 13, 2024 08:53
@seunghye218 seunghye218 self-assigned this Sep 13, 2024
Copy link

@jongmee jongmee left a comment

Choose a reason for hiding this comment

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

안녕하세요 마크! 즐거운 한가위 보내고 계신가요 ☺️

학습테스트 포함해서 간단하게 리뷰 몇 개 남겼습니다 :) 고생하셨어요 !

@@ -3,8 +3,8 @@ handlebars:

server:
tomcat:
accept-count: 1
max-connections: 1
accept-count: 0
Copy link

Choose a reason for hiding this comment

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

accept-count를 0으로 변경한 이유가 궁금해요 ! 🙋🏻‍♀️

Copy link
Author

Choose a reason for hiding this comment

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

테스트에서 명확하게 2개의 요청만 처리하기위해 설정했어요!
tomcat 공식문서를 통해 accept-count 를 listen 함수의 backlog 인자라고 이해를 했어요. 0으로 설정함으로써 이미 max-connections 만큼 연결됐을 때 추가 연결 요청을 무시할 수 있다고 생각했어요.

하지만

The operating system may ignore this setting and use a different size for the queue.
운영 체제는 이 설정을 무시하고 대기열에 다른 크기를 사용할 수 있습니다.

라고 명시되어 있기 때문에 맥 운영체제에서 정확히 어떻게 동작할지는 잘 모르겠네요 😅

Copy link

Choose a reason for hiding this comment

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

오호 앞선 2개의 요청이 끝나도 추가 요청이 들어오는 일이 없게끔 하신거군요 ! 👍

@@ -1,5 +1,6 @@
package thread.stage2;
Copy link

Choose a reason for hiding this comment

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

stage1의 학습테스트도 LMS에 기재된 것처럼 실패하도록 실행시켜 본 후, 통과하도록 해보는 것도 있으면 좋겠네요 ! 😊

Copy link
Author

Choose a reason for hiding this comment

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

스레드 강의 끝나고 팀원들과 같이 진행해보았는데 추가 코드를 말씀하신걸까요?

Copy link

Choose a reason for hiding this comment

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

스레드 기준으로 디버거를 돌려도 테스트에 통과할 수 있도록 학습해보면 좋을 것 같다는 의도였어요 ! (커밋 내역에 없어서 여기까지는 안 해보신 줄 알고 😅) 팀원들과 같이 진행해보셨다니! 좋네요 👍

Copy link

@jongmee jongmee left a comment

Choose a reason for hiding this comment

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

답변 잘 읽었습니다 ! 이번 미션 전체적으로 공식 문서를 읽으며 열심히 공부하신 것 같아서 멋져요 👍

고생하셨습니다 ☺️

@@ -1,5 +1,6 @@
package thread.stage2;
Copy link

Choose a reason for hiding this comment

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

스레드 기준으로 디버거를 돌려도 테스트에 통과할 수 있도록 학습해보면 좋을 것 같다는 의도였어요 ! (커밋 내역에 없어서 여기까지는 안 해보신 줄 알고 😅) 팀원들과 같이 진행해보셨다니! 좋네요 👍

@@ -3,8 +3,8 @@ handlebars:

server:
tomcat:
accept-count: 1
max-connections: 1
accept-count: 0
Copy link

Choose a reason for hiding this comment

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

오호 앞선 2개의 요청이 끝나도 추가 요청이 들어오는 일이 없게끔 하신거군요 ! 👍

@jongmee jongmee merged commit 6960507 into woowacourse:seunghye218 Sep 19, 2024
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.

3 participants