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

[BE] 팀채팅시 이벤트 발행에 db조회 제거 #956

Merged
merged 2 commits into from
Mar 12, 2024

Conversation

pilyang
Copy link
Collaborator

@pilyang pilyang commented Mar 12, 2024

PR 내용

  • 채팅으로 인한 sse 이벤트 발행시 채팅 내용 조회를 위한 db조회 제거

초기 db조회 없이 발행을 하도록 하였다가 코드의 일관성 (domain event) 을 위해 이벤트에는 내용등 정보 없이 이벤트 발행시 최초 1회 다시 db조회를 하도록 변경

해당 기능 구현 이후 db조회로 인한 채팅 성능의 하락이 눈에 띌 정도로 생김

일단은 사용자들의 불편함 없는 경험이 중요하다고 생각하여 db조회 없이 피드 sse 를 발행할 수 있도록 재 변경 진행

참고자료

의논할 거리

Copy link

Test Results

153 files  153 suites   20s ⏱️
500 tests 500 ✅ 0 💤 0 ❌
504 runs  504 ✅ 0 💤 0 ❌

Results for commit 68d919a.

@pilyang pilyang merged commit 9ec31a8 into develop Mar 12, 2024
3 checks passed
@pilyang pilyang deleted the refactor/be/team-chat-optimize branch March 12, 2024 10:19
Copy link

@iamjooon2 iamjooon2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants