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

Hotfix: 토픽 조회시 순서 보장하도록 수정 #519

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

cpot5620
Copy link
Collaborator

작업 대상

Topic

📄 작업 내용

Topic 클래스의 자료구조를 List로 변경하였습니다.
기존에 multiple Bag(?) 오류로 인해, Set을 사용하였는데 이 경우, 데이터베이스에서 데이터를 읽어올 때, 순서를 보장하지 않게 됩니다.
어플리케이션 단에서 순서가 보장되지 않아서 문제가 발생하는 것이 아니라, 데이터베이스에서 읽어올 때 발생하는 문제입니다.
이에 따라, 기존 매튜의 hotfix로 문제가 해결되지 않았던 것입니다.

🙋🏻 주의 사항

성능 개선 과정에서, Bookmark count를 컬럼으로 추가함에 따라 기존 @EntityGraph 사용시 함께 fetch join 해오던 Bookmarks를 제거하였습니다.
이로 인해, mutliple bag 관련 예외가 발생하지 않으며, 데이터의 기존 순서 또한 보장될 수 있습니다.

참고 사항
별개로, 현재 프론트엔드에서 토픽을 상세조회할 때 2번의 API 요청이 발생하고 있습니다.
또한, 토픽 내의 핀의 정보를 클릭할 때마다, 토픽 상세 조회 API를 호출하고 있습니다.
이 부분은 세인 및 아이크에게 전달된 사항입니다.

스크린샷

📎 관련 이슈

레퍼런스

@cpot5620 cpot5620 added bug Something isn't working BE 백엔드 관련 이슈 우선순위 : 상 labels Sep 27, 2023
@github-actions
Copy link

Unit Test Results

  67 files    67 suites   22s ⏱️
288 tests 288 ✔️ 0 💤 0
299 runs  299 ✔️ 0 💤 0

Results for commit 41ff3b0.

Copy link
Collaborator

@kpeel5839 kpeel5839 left a comment

Choose a reason for hiding this comment

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

맞네요오오 저희가 bookmark count 를 추가했는데도 entity graph 에서 bookmarks 를 제거하지 않는 실수를 했군요~! 고생하셨어요 쥬니~~

Copy link
Collaborator

@junpakPark junpakPark left a comment

Choose a reason for hiding this comment

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

수고하샸습니다 👍👍

@kpeel5839 kpeel5839 merged commit e75517c into main Sep 27, 2023
3 checks passed
@yoondgu yoondgu added this to the 최종 데모데이 milestone Oct 3, 2023
@yoondgu yoondgu deleted the hotfix/pin-order branch October 6, 2023 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈 bug Something isn't working 우선순위 : 상
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants