Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/#143 채팅 요약 정보가 제대로 집계되지 않는 문제 수정 #144

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

coke98
Copy link
Contributor

@coke98 coke98 commented Aug 22, 2024

🤨 Motivation

🔑 Key Changes

  • 채팅 요약 정보 집계시 aggregate 구문 roomId 필드 추가
  • 채팅 요약 정보 집계시 aggregate 구문 unreadCount 필드 -> numberOfUnreadMessages 필드로 수정

🙏 To Reviewers

aggregate를 사용할때, 필드 명시가 아닌 Aggregation.group("roomId")(단순 그룹화)으로만 명시했던터라 애초에 roomId를 필드까지 가지고 오지 못하는 것이 맞았네요..
as("unreadCount");도 현재 dto 필드와 동일하지 못해 발생한 정말 간단한 문제였어요
왜 몰랐을까 싶은데.. 이런걸 캐치 못한건 스스로도 채찍질이 필요해보이네요
복잡한 쿼리를 관리를 위해 aggregate문을 처음 도입할때 제대로 이해하지못한게 가장 컸던 것 같습니다. 앞으로는 처음 사용해보는 구문도 제대로 이해 후 활용하도록 하겠습니다 🥹

간단한 문제일거라고는 짐작했고.. 집계 코드 쪽 원인을 찾고부터는 진짜 단순한 문제라 바로 해결했는데, 원인 파악을 위해 로컬 매칭 + 디버깅 환경 구성까지 시간이 조금 걸렸던 것 같습니다.

😇 매칭 환경 구성을 도와주신 @suhyun0918 수현님께 정말 감사드립니다!🙇‍♂️😭✨ 덕분에 바로 끝낼 수 있었네요👍

@coke98 coke98 self-assigned this Aug 22, 2024
@coke98 coke98 merged commit e3b3c2d into develop Aug 22, 2024
1 check passed
@mjj111 mjj111 added the fix 오류 관련 태그 label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 오류 관련 태그
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants