2조 코드 리뷰

이지표

• 디자인이 다들 너무 예쁘십니다!
• 하나의 connection으로 모든 처리를 하는 점 재밌었습니다.
  • 팀원분들이랑 이야기를 나누어보니 connection에 시간 제한이 있다고 합니다!
• 예외처리를 filter, xml설정 등등 다양한 방법을 보게 되어 좋았습니다.
• 쿼리를 자동으로 생성해주는거 멋지네여!

지찬우

• 테스트 커버리지 90% 이상 멋집니다..
• 리플렉션을 활용해 어노테이션 기반 인증 필터를 적용한 부분이 인상깊었습니다.
• 커스텀 validation 어노테이션을 만들어 request body를 검증하는 부분이 인상깊었습니다.
• 공통 Servlet 객체를 만들어 모든 서블릿이 해당 공통 서블릿을 상속하도록 하여 의존성 주입이 가능하도록 구현한 부분이 인상깊었습니다.
• 하나의 connection을 여러 스레드가 점유하여 사용하면 발생하는 문제에 대해 스쿼드 멤버들과 토론했는데 재미있었습니다!

윤중진

• 다들 디자인이 너무 이쁘네요.
• 예외 처리 부분에서 jsp, json 부분을 나눠서 처리하는 부분이 좋습니다.
• 서블릿을 구현해서 직접 매핑 정보를 관리하고, 생성자 주입을 구현하는 부분이 좋았습니다.
• DB Connection pool을 구현하려고 한 점이 좋네요.
  • 다들 의욕이 대단하신 것 같습니다.. 본 받아야겠습니다.

박정제

• 다음 글, 이전 글 기능과 조회수 기능 너무 마음에 들어요
• 테스트 하기 까다로운 옵션들이 많았는데 목객체들을 만들어서 테스트를 다 하는부분 잘 배웠습니다
• JPA를 만들어서 데이터베이스를 구현한점 좋았습니다
• 커넥션을 한개를 공유해서 사용하면 무슨 문제가 될까?
• Locust
• 너무 많은 데이터는 랭더링 자체가 문제가 될 수 있다

오민석

• 커넥션 풀을 해야하는 이유를 같이 알아보는 시간을 가졌습니다 재밌고 브라우저가 많은 렌더링을 해주지 않는다는 것을 알았습니다
• 디자인까지 신경쓰셨는데 저는 안해서… 다들 존경합니다!
• 리플렉션을 활용해서 로그인 인증 필터를 구현하신게 인상깊었습니다

김현수

• 💡 새로 알 게 된 점
  • @NotNull 어노테이션을 통해 벨리데이션을 하면 좋다~
  • 필터로 RESTAPI 요청에 대한 Exception 을 묶어서 처리한 점이 좋아 보였다.
    • 나는 하나씩 잡아서 처리해준다.
  • deleted_at 으로 삭제 여부 확인
  • username을 넣지 않고 조인해서 넣도록 수정
• 🚨 궁금한 점
  • 제가 DB connection 을 하나만 두고 있는데 유저 2만명일 때까지 문제가 발생하지 않습니다.
    • Connection Pool 을 써야하는 명확한 이유가 무엇인가요 ?
  • 그리고 동시에 쿼리를 날려도 처리를 하는데 어떻게 된거죠 ?
    • 쿼리 실행 속도가 너무 빨라서 그런거 같다.