[Chat] Redis Stream 기반 채팅 메세지 저장&조회 by heygeeji · Pull Request #204 · prgrms-web-devcourse-final-project/WEB7_9_codecrete_BE

heygeeji · 2025-12-28T18:36:02Z

🔗 관련 이슈

관련된 이슈 번호를 연결해주세요.
Close #이슈번호 를 쓰면 PR merge 시 자동으로 close 됩니다.

Close [Chat] Redis Stream기반 채팅 메시지 관리 #185

🚀 PR 개요

이 PR이 어떤 변경을 포함하고 있는지 간단히 설명해주세요.

Redis Stream 기반 채팅 메세지 저장
Redis Stream 기반 채팅 메세지 조회
채팅하는 유저 캐싱 처리

📌 변경 사항

주요 변경 내용을 체크리스트 형태로 정리해주세요.

🧪 테스트 방법

변경된 내용을 어떻게 테스트했는지 구체적으로 적어주세요.

📸 스크린샷 (선택)

UI 변경 또는 시각적으로 확인할 수 있는 변경이 있다면 첨부해주세요.

⚠️ 참고 사항

리뷰어가 알아야 할 사항이 있다면 자유롭게 작성해주세요.

github-actions · 2025-12-28T18:37:59Z

Test Results

6 tests ±0 6 ✅ ±0 0s ⏱️ ±0s
3 suites ±0 0 💤 ±0
3 files ±0 0 ❌ ±0

Results for commit 99980b6. ± Comparison against base commit ddce6fe.

♻️ This comment has been updated with latest results.

github-actions · 2025-12-28T18:38:16Z

⭐ JaCoCo Coverage

Line Coverage: 5.30%

📄 Coverage Details

Overall Line Coverage: 5.27% (204 covered / 3872 lines)

Package Summary (lowest first)

Package	Line %	Covered	Missed
`com/back/web7_9_codecrete_be/domain/plans/controller`	0.00%	0	49
`com/back/web7_9_codecrete_be/domain/location/controller`	0.00%	0	54
`com/back/web7_9_codecrete_be/domain/community/post/service`	0.00%	0	45
`com/back/web7_9_codecrete_be/global/rsData`	0.00%	0	11
`com/back/web7_9_codecrete_be/domain/chats/entity`	0.00%	0	4
`com/back/web7_9_codecrete_be/domain/location/entity`	0.00%	0	15
`com/back/web7_9_codecrete_be/global/scheduler`	0.00%	0	11
`com/back/web7_9_codecrete_be/domain/auth/entity`	0.00%	0	0
`com/back/web7_9_codecrete_be/global/websocket`	0.00%	0	10
`com/back/web7_9_codecrete_be/domain/auth/service`	0.00%	0	135
`com/back/web7_9_codecrete_be/domain/plans/service`	0.00%	0	362
`com/back/web7_9_codecrete_be/global/initData`	0.00%	0	54
`com/back/web7_9_codecrete_be/domain/users/controller`	0.00%	0	26
`com/back/web7_9_codecrete_be/domain/email/entity`	0.00%	0	6
`com/back/web7_9_codecrete_be/domain/community/comment/service`	0.00%	0	17
`com/back/web7_9_codecrete_be/domain/auth/controller`	0.00%	0	24
`com/back/web7_9_codecrete_be/domain/community/comment/controller`	0.00%	0	7
`com/back/web7_9_codecrete_be/domain/artists/controller`	0.00%	0	34
`com/back/web7_9_codecrete_be/domain/community/comment/entity`	0.00%	0	12
`com/back/web7_9_codecrete_be/domain/community/post/controller`	0.00%	0	20
`com/back/web7_9_codecrete_be/global/aspect`	0.00%	0	4
`com/back/web7_9_codecrete_be/global/wikidata`	0.27%	1	374
`com/back/web7_9_codecrete_be/global/musicbrainz`	0.46%	1	215
`com/back/web7_9_codecrete_be/domain/artists/service`	0.54%	4	740
`com/back/web7_9_codecrete_be/domain/users/service`	0.96%	1	103
`com/back/web7_9_codecrete_be/domain/email/service`	1.61%	1	61
`com/back/web7_9_codecrete_be/global/flo`	1.85%	1	53
`com/back/web7_9_codecrete_be/domain/chats/service`	2.27%	2	86
`com/back/web7_9_codecrete_be/domain/concerts/controller`	2.50%	1	39
`com/back/web7_9_codecrete_be/domain/concerts/entity`	3.26%	3	89
`com/back/web7_9_codecrete_be/domain/concerts/repository`	3.45%	1	28
`com/back/web7_9_codecrete_be/global/storage`	3.57%	2	54
`com/back/web7_9_codecrete_be/domain/concerts/service`	4.78%	25	498
`com/back/web7_9_codecrete_be/domain/artists/entity`	6.45%	4	58
`com/back/web7_9_codecrete_be/global/rq`	9.80%	5	46
`com/back/web7_9_codecrete_be/domain/users/util`	14.29%	1	6
`com/back/web7_9_codecrete_be/domain/chats/controller`	16.67%	2	10
`com/back/web7_9_codecrete_be/domain/community/post/entity`	16.67%	6	30
`com/back/web7_9_codecrete_be/domain/plans/entity`	21.88%	21	75
`com/back/web7_9_codecrete_be/domain/users/entity`	23.08%	12	40
`com/back/web7_9_codecrete_be/global/security`	27.62%	29	76
`com/back/web7_9_codecrete_be/global/spotify`	35.71%	5	9
`com/back/web7_9_codecrete_be/domain/location/service`	43.07%	59	78
`com/back/web7_9_codecrete_be/global/doc`	100.00%	17	0

Lowest Covered Classes (Top 20)

Class	Line %	Missed
`com.back.web7_9_codecrete_be.domain.plans.service.PlanService`	0.00%	362
`com.back.web7_9_codecrete_be.domain.artists.service.EnrichStepExecutor`	0.00%	130
`com.back.web7_9_codecrete_be.domain.artists.service.ArtistService`	0.00%	98
`com.back.web7_9_codecrete_be.domain.artists.service.WikidataEnrichHelper`	0.00%	78
`com.back.web7_9_codecrete_be.domain.auth.service.AuthService`	0.00%	73
`com.back.web7_9_codecrete_be.global.initData.BaseInitData`	0.00%	54
`com.back.web7_9_codecrete_be.domain.plans.controller.PlanController`	0.00%	49
`com.back.web7_9_codecrete_be.domain.location.controller.KakaoApiController`	0.00%	43
`com.back.web7_9_codecrete_be.domain.concerts.entity.Concert`	0.00%	42
`com.back.web7_9_codecrete_be.domain.artists.entity.Artist`	0.00%	40
`com.back.web7_9_codecrete_be.domain.chats.service.ChatPolicyService`	0.00%	38
`com.back.web7_9_codecrete_be.domain.plans.entity.Schedule`	0.00%	37
`com.back.web7_9_codecrete_be.global.security.JwtTokenProvider`	0.00%	37
`com.back.web7_9_codecrete_be.domain.community.post.service.PostService`	0.00%	36
`com.back.web7_9_codecrete_be.domain.users.entity.User`	0.00%	31
`com.back.web7_9_codecrete_be.domain.location.service.LocationService`	0.00%	30
`com.back.web7_9_codecrete_be.domain.concerts.entity.ConcertPlace`	0.00%	29
`com.back.web7_9_codecrete_be.domain.plans.entity.Plan`	0.00%	27
`com.back.web7_9_codecrete_be.domain.artists.controller.ArtistsController`	0.00%	27
`com.back.web7_9_codecrete_be.domain.auth.service.TokenService`	0.00%	26

Changed Classes (from this PR)

Source (PR)	Class Prefix	Line %	Covered	Missed
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/service/ChatPolicyService.java`	`com.back.web7_9_codecrete_be.domain.chats.service.ChatPolicyService`	0.00%	0	38
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/service/ChatRoomService.java`	`com.back.web7_9_codecrete_be.domain.chats.service.ChatRoomService`	0.00%	0	15
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/service/ChatMessageReadService.java`	`com.back.web7_9_codecrete_be.domain.chats.service.ChatMessageReadService`	0.00%	0	5
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/controller/ChatMessageReadController.java`	`com.back.web7_9_codecrete_be.domain.chats.controller.ChatMessageReadController`	0.00%	0	2
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/controller/ChatRoomController.java`	`com.back.web7_9_codecrete_be.domain.chats.controller.ChatRoomController`	0.00%	0	2
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/dto/request/ChatMessageRequest.java`	`com.back.web7_9_codecrete_be.domain.chats.dto.request.ChatMessageRequest`	0.00%	0	0
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/dto/response/ChatMessageResponse.java`	`com.back.web7_9_codecrete_be.domain.chats.dto.response.ChatMessageResponse`	0.00%	0	0
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/dto/response/ChatReadResponse.java`	`com.back.web7_9_codecrete_be.domain.chats.dto.response.ChatReadResponse`	0.00%	0	0
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/dto/response/ChatUserCache.java`	`com.back.web7_9_codecrete_be.domain.chats.dto.response.ChatUserCache`	0.00%	0	0
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/repository/ChatStreamRepository.java`	`com.back.web7_9_codecrete_be.domain.chats.repository.ChatStreamRepository`	0.00%	0	0
`src/main/java/com/back/web7_9_codecrete_be/domain/users/service/UserService.java`	`com.back.web7_9_codecrete_be.domain.users.service.UserService`	1.00%	1	99
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/service/ChatUserCacheService.java`	`com.back.web7_9_codecrete_be.domain.chats.service.ChatUserCacheService`	6.25%	1	15
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/service/ChatMessageService.java`	`com.back.web7_9_codecrete_be.domain.chats.service.ChatMessageService`	7.14%	1	13
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/controller/ChatMessageController.java`	`com.back.web7_9_codecrete_be.domain.chats.controller.ChatMessageController`	20.00%	1	4
`src/main/java/com/back/web7_9_codecrete_be/domain/chats/controller/ChatStompDocsController.java`	`com.back.web7_9_codecrete_be.domain.chats.controller.ChatStompDocsController`	33.33%	1	2
`src/main/java/com/back/web7_9_codecrete_be/global/security/SecurityConfig.java`	`com.back.web7_9_codecrete_be.global.security.SecurityConfig`	100.00%	25	0

🔗 Full HTML report: See artifact jacoco-full-html on this run → https://github.com/prgrms-web-devcourse-final-project/WEB7_9_codecrete_BE/actions/runs/20558018198

larama-C · 2025-12-29T07:39:57Z

 		LocalDateTime ticketTime = concert.getTicketTime();
 		if (ticketTime == null) {
-			return Duration.ofMinutes(0);
+			return Duration.ofMinutes(10);


티켓 타임이 존재하지 않는 경우 왜 10분을 반환하는지 알 수 있을까요?

ticketTime이 없는 경우 TTL을 0으로 두면 매 요청마다 DB 조회가 발생하므로,
불필요한 DB 부하를 방지하고 추후 ticketTime 저장 시 정책이 반영되도록 TTL을 10분으로 했습니다!

아하 그런 이유였군요 감사합니다!

Creamcheesepie · 2025-12-29T07:42:03Z

+	private final UserRepository userRepository;
+
+	private String cacheKey(String email) {
+		return "chat:user:" + email;


email외에 짧은 식별값을 쓰는 것도 좋을 것 같습니다!

Creamcheesepie · 2025-12-29T07:44:31Z

+	private String exclusiveBefore(String messageId) {
+		String[] parts = messageId.split("-");
+		long time = Long.parseLong(parts[0]); // 밀리초 타임스탬프
+		long seq = Long.parseLong(parts[1]); // 같은 밀리초 내 순번


같은 시간 내에서도 순번을 정하신것 좋은 방법인 것 같습니다

Creamcheesepie

무난하고 깔끔해 보입니다.

ys0221

고생하셨습니다!!

kms152000

수고하셨습니다~

tobbot16

수고하셨습니다!

heygeeji added 8 commits December 29, 2025 02:51

docs: 채팅방 description 수정

cf21973

chore: WebSocket 엔드포인트 수정

7659cb9

feat: 채팅 메세지 Redis Stream에 저장

0e030b3

refactor: dto 패키지 변경

a7af6ed

refactor: 채팅 정책 TTL 조정 및 문서화 개선

63d8025

feat: Redis Stream 기반 채팅 메시지 조회 구현

65711fc

feat: 채팅 메시지 전송 시 유저 정보 Redis 캐싱 적용

a1b851e

feat: 유저 닉네임 변경 시 채팅 유저 정보 캐시 무효화 처리

99980b6

heygeeji linked an issue Dec 28, 2025 that may be closed by this pull request

[Chat] Redis Stream기반 채팅 메시지 관리 #185

Closed

heygeeji force-pushed the feat/#185 branch from f1f1890 to 99980b6 Compare December 28, 2025 18:44

larama-C assigned heygeeji Dec 29, 2025

larama-C added the Feature label Dec 29, 2025

larama-C reviewed Dec 29, 2025

View reviewed changes

Creamcheesepie reviewed Dec 29, 2025

View reviewed changes

Creamcheesepie approved these changes Dec 29, 2025

View reviewed changes

ys0221 approved these changes Dec 29, 2025

View reviewed changes

larama-C approved these changes Dec 29, 2025

View reviewed changes

kms152000 approved these changes Dec 29, 2025

View reviewed changes

tobbot16 approved these changes Dec 29, 2025

View reviewed changes

Creamcheesepie merged commit cf3da3c into main Dec 29, 2025
2 checks passed

Creamcheesepie deleted the feat/#185 branch December 29, 2025 08:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Chat] Redis Stream 기반 채팅 메세지 저장&조회#204

[Chat] Redis Stream 기반 채팅 메세지 저장&조회#204
Creamcheesepie merged 8 commits into
mainfrom
feat/#185

heygeeji commented Dec 28, 2025

Uh oh!

github-actions Bot commented Dec 28, 2025 •

edited

Loading

Uh oh!

github-actions Bot commented Dec 28, 2025 •

edited

Loading

Uh oh!

larama-C Dec 29, 2025

Uh oh!

heygeeji Dec 29, 2025

Uh oh!

larama-C Dec 29, 2025

Uh oh!

Creamcheesepie Dec 29, 2025

Uh oh!

Creamcheesepie Dec 29, 2025

Uh oh!

Creamcheesepie left a comment

Uh oh!

ys0221 left a comment

Uh oh!

kms152000 left a comment

Uh oh!

tobbot16 left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

heygeeji commented Dec 28, 2025

🔗 관련 이슈

🚀 PR 개요

📌 변경 사항

🧪 테스트 방법

📸 스크린샷 (선택)

⚠️ 참고 사항

Uh oh!

github-actions Bot commented Dec 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

github-actions Bot commented Dec 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⭐ JaCoCo Coverage

📄 Coverage Details

Uh oh!

larama-C Dec 29, 2025

Choose a reason for hiding this comment

Uh oh!

heygeeji Dec 29, 2025

Choose a reason for hiding this comment

Uh oh!

larama-C Dec 29, 2025

Choose a reason for hiding this comment

Uh oh!

Creamcheesepie Dec 29, 2025

Choose a reason for hiding this comment

Uh oh!

Creamcheesepie Dec 29, 2025

Choose a reason for hiding this comment

Uh oh!

Creamcheesepie left a comment

Choose a reason for hiding this comment

Uh oh!

ys0221 left a comment

Choose a reason for hiding this comment

Uh oh!

kms152000 left a comment

Choose a reason for hiding this comment

Uh oh!

tobbot16 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

github-actions Bot commented Dec 28, 2025 •

edited

Loading

github-actions Bot commented Dec 28, 2025 •

edited

Loading