Merged
Conversation
|
Caution Review failedThe pull request is closed. 둘러보기이 PR은 댓글 기능에 Redis 캐싱을 도입하고 응답 구조를 개선합니다. 새로운 변경사항
예상 코드 리뷰 노력도🎯 3 (Moderate) | ⏱️ ~25 minutes 관련 PR
시
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
12 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣ Issue Number
📝 요약(Summary)
댓글 도메인에 Redis 캐시를 적용하기 위해 응답 구조와 서비스 시그니처를 정리했습니다.
List→CommentListResponse로 래핑하여 캐시 안정성 확보commentList) 및 댓글 수(commentCount)에 Redis 캐시 적용commentList캐시에 대해 전용 Serializer 및 TTL 설정 추가🛠️ PR 유형
어떤 변경 사항이 있나요?
📸스크린샷 (선택
💬 공유사항 to 리뷰어
List<CommentResponse>대신CommentListResponse로 래핑하여 직렬화/역직렬화 타입을 명확히 했습니다.commentList,commentCount를 함께 evict 하여캐시 불일치가 발생하지 않도록 처리했습니다.
commentCount는 단순 타입(Long)이므로 별도 CacheConfig 없이 기본 설정을 사용했습니다.✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.
Summary by CodeRabbit
릴리스 노트