feat : 알림 목록 조회 성능 개선 #177
Merged
Conversation
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
- 발생하는 오버헤드에 비해 얻을 수 있는 성능적 이점이 적은 캐시를 제거함 - 대신 mongodb에 복합 인덱스를 적용하고, TTL(6개월)을 설정했다. - 파이썬 스크립트로 데이터를 삽입하고 프론트 페이지에서 성능을 측정했다.
WalkthroughRedis 캐싱 관련 구성, 서비스, 설정 코드가 모두 제거되었습니다. NotificationDocument에 MongoDB 복합 인덱스와 만료 인덱스가 추가되었습니다. chat-page.html에 알림 요청 및 지연 측정 기능이 도입되었습니다. 테스트용 MongoDB 알림 데이터를 생성하는 Python 스크립트가 새로 추가되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Browser
participant Server
participant MongoDB
User->>Browser: requestNotifications() 호출
Browser->>Server: GET /api/notifications (with Auth)
Server->>MongoDB: 알림 목록 쿼리
MongoDB-->>Server: 알림 목록 반환
Server-->>Browser: 알림 목록 응답
Browser->>Browser: wsLatencyStart로 지연 측정
Server-->>Browser: WebSocket으로 알림 전송
Browser->>Browser: wsLatencyStart로 round-trip latency 계산 및 로그 출력
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
src/test/resources/send_notifications.py (1)
21-24: 테스트 데이터 다양성 개선 제안현재 이메일 생성 로직이 단순하여 실제 사용자 분포를 제대로 시뮬레이션하지 못할 수 있습니다.
더 현실적인 테스트 데이터를 위해 다음과 같이 개선할 수 있습니다:
+import string + def random_email(): - # 1부터 10까지 랜덤, 1일 때는 [email protected] - idx = random.randint(1, 10) - return "[email protected]" if idx == 1 else f"ttest{idx}@test.com" + # 더 다양한 이메일 패턴 생성 + if random.random() < 0.1: # 10% 확률로 고정 이메일 + return "[email protected]" + + username = ''.join(random.choices(string.ascii_lowercase, k=random.randint(5, 10))) + domains = ["test.com", "example.org", "demo.net"] + domain = random.choice(domains) + return f"{username}@{domain}"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/main/java/org/ezcode/codetest/infrastructure/notification/config/NotificationRedisCacheConfig.java(0 hunks)src/main/java/org/ezcode/codetest/infrastructure/notification/model/NotificationDocument.java(3 hunks)src/main/java/org/ezcode/codetest/infrastructure/notification/service/NotificationService.java(2 hunks)src/main/resources/templates/chat-page.html(2 hunks)src/test/resources/send_notifications.py(1 hunks)
💤 Files with no reviewable changes (1)
- src/main/java/org/ezcode/codetest/infrastructure/notification/config/NotificationRedisCacheConfig.java
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: chat26666
PR: ezcode-my/backend#37
File: src/main/java/org/ezcode/codetest/infrastructure/elasticsearch/repository/ProblemElasticsearchRepositoryDsl.java:10-10
Timestamp: 2025-06-06T19:56:27.759Z
Learning: When user chat26666 mentions that code was implemented in a previous PR but explanation was missing, they may submit a follow-up PR just to add documentation/explanation without the actual implementation code.
📚 Learning: in the ezcode-my/backend codebase, skills are managed by admin user chat26666 and skill changes happ...
Learnt from: chat26666
PR: ezcode-my/backend#74
File: src/main/java/org/ezcode/codetest/infrastructure/persistence/repository/game/mysql/skill/SkillRepositoryImpl.java:45-60
Timestamp: 2025-06-18T15:25:27.923Z
Learning: In the ezcode-my/backend codebase, skills are managed by admin user chat26666 and skill changes happen infrequently. Therefore, using `@CacheEvict(value = "skill", allEntries = true)` in SkillRepositoryImpl is acceptable despite the cache rebuild cost, as the performance impact is minimal due to low update frequency.
Applied to files:
src/main/java/org/ezcode/codetest/infrastructure/notification/service/NotificationService.java
🔇 Additional comments (7)
src/main/java/org/ezcode/codetest/infrastructure/notification/model/NotificationDocument.java (2)
24-26: 우수한 인덱스 설계
principalName과createdAt필드에 대한 복합 인덱스는 알림 목록 조회 성능을 크게 향상시킬 것입니다. 사용자별 최신 알림 조회 패턴에 최적화된 설계입니다.
41-41: 적절한 데이터 보관 정책180일 TTL 설정으로 약 6개월간 알림 데이터를 보관하는 것은 합리적입니다. 자동 정리를 통해 데이터베이스 크기를 관리하고 성능을 유지할 수 있습니다.
src/main/resources/templates/chat-page.html (2)
188-204: 우수한 성능 측정 구현WebSocket을 통한 알림 요청-응답 지연시간 측정 기능이 잘 구현되었습니다.
performance.now()를 사용한 정확한 시간 측정과 콘솔에서 직접 호출 가능한 전역 함수 노출이 개발자 경험을 향상시킵니다.
227-231: 정확한 지연시간 계산WebSocket 콜백에서 라운드트립 지연시간을 정확히 계산하고 로깅하는 구현이 성능 분석에 도움이 될 것입니다. 측정 후
wsLatencyStart를 0으로 리셋하는 것도 적절합니다.src/main/java/org/ezcode/codetest/infrastructure/notification/service/NotificationService.java (3)
24-28: 깔끔한 의존성 정리Redis 캐시 의존성을 완전히 제거하고 MongoDB 저장소만 남긴 것이 PR 목표와 일치합니다. 캐시 오버헤드 대비 성능 이득이 미미하다는 분석 결과를 반영한 적절한 결정입니다.
30-46: MongoDB 직접 조회로 단순화
@Cacheable어노테이션을 제거하고 MongoDB에서 직접 조회하도록 변경한 것이 올바릅니다. 복합 인덱스 추가와 함께 충분한 성능을 제공할 것으로 예상됩니다.
48-61: 캐시 무효화 로직 제거 확인알림 생성과 읽음 처리에서 캐시 무효화 로직이 완전히 제거되어 코드가 단순해졌습니다. MongoDB 인덱스 기반 성능 최적화 전략과 일관성 있는 변경입니다.
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.
작업 내용
참고 사항
Summary by CodeRabbit
신규 기능
버그 수정
성능 개선