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.

Sign up for GitHub

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

Jump to bottom

채팅방 목록 조회 시 쿼리 변경 #751

Closed
wants to merge 1 commit into from
Closed

채팅방 목록 조회 시 쿼리 변경 #751

wants to merge 1 commit into from

Conversation

apptie
Copy link
Collaborator

@apptie apptie commented Nov 17, 2023

📄 작업 내용 요약

  • 채팅방 목록 조회 시 쿼리 변경
    • 서브 쿼리에서 GROUP BY 제거

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

변경 내용은 간단하지만 실행 계획쪽이나 뭐 그런쪽과 연관이 되어 있습니다
디스커션 참고하시면 될 것 같습니다

📎 Issue 번호

@apptie apptie added refactor 기존 기능에 변경이 없는 구현 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 suggestion 코드 변경 사항이 있는 경우 팀원들에게 제안하기 위함 labels Nov 17, 2023
@apptie apptie self-assigned this Nov 17, 2023
kwonyj1022
kwonyj1022 approved these changes Nov 17, 2023
View reviewed changes
Copy link
Collaborator

@kwonyj1022 kwonyj1022 left a comment

Choose a reason for hiding this comment

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

감사합니다!

swonny
swonny approved these changes Nov 19, 2023
View reviewed changes
Copy link
Collaborator

@swonny swonny left a comment

Choose a reason for hiding this comment

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

지토 진짜 대단하네요....
이런 지식은 도대체 어떻게 알게되는건가요..
한 줄 수정에도 깊게 고민하신 것 같아 멋집니다 👍🏻

JJ503
JJ503 approved these changes Nov 19, 2023
View reviewed changes
Copy link
Member

@JJ503 JJ503 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

이런 부분들까지 확인하셨다니 대단해요 지토..! 👍
디스커션 덕분에 group by가 필요 없음은 물론, 성능까지 끌어올릴 수 있다는 점에 대해 잘 이해할 수 있었습니다. 감사합니다!

@apptie apptie closed this Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kwonyj1022 kwonyj1022 kwonyj1022 approved these changes

@swonny swonny swonny approved these changes

@JJ503 JJ503 JJ503 approved these changes

Assignees

@apptie apptie

Labels
backend 백엔드와 관련된 이슈나 PR에 사용 refactor 기존 기능에 변경이 없는 구현 변경 시 suggestion 코드 변경 사항이 있는 경우 팀원들에게 제안하기 위함
Projects
2023-3-ddang
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@apptie @JJ503 @swonny @kwonyj1022