[Refactor] #294 - Redis 유저 조회 로직 개선

[its-sky]

🚀PullRequest🚀

📟 관련 이슈

• Resolved: [Refactor] Redis 유저 조회 로직 개선 #294

💻 작업 내용

• Security 로그인 로직 중에서 UserDetails에 유저를 Redis에서 조회하여 주입하고 이를 커스텀 어노테이션에서 사용하는 로직이 존재하였습니다. 하지만 Access Token의 userId만을 주입하여 사용하여도 충분히 문제없는 로직이었습니다. 따라서 Redis에서 유저를 조회하여 UserDetails에 삽입하는 로직을 제거하고 JMeter로 성능 테스트를 진행하였습니다.
• JwtTokenProvider와 UserService에서 제대로 동작하지 않거나 불필요한 검증 로직 및 조회 로직을 삭제하였습니다.
• UserService에서 사용되는 userService.findById 를 userService.findByIdWithCache를 이용하여 Redis에 캐싱되어 있는 유저 정보를 사용하여 처리하도록 하였습니다.

📝 리뷰 노트

[JMeter 성능 테스트 - 기존 Redis 로직이었을 때 in Local]

[JMeter 성능 테스트 - Redis 조회 로직을 제거했을 때 in Local]

로컬에서 테스트를 진행하였을 때 조회 API 10만 건에서 평균 13ms에서 11ms로 개선되었음을 확인하였습니다. 저희 서버 인프라 상 싱글 코어인만큼 성능 차이는 더 확연히 벌어질 것이라고 생각합니다.

모르는 부분이 있거나 이상한 점은 코드리뷰에 달아주세요 감사합니다 :)

[0lynny]

어푸룹 했습니다 !