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] Refactor/#494 성능 개선을 위한 조회 쿼리 수정 #508

Merged
merged 2 commits into from
Sep 27, 2023

Conversation

yoondgu
Copy link
Collaborator

@yoondgu yoondgu commented Sep 27, 2023

작업 대상

성능 개선을 위해 조회 쿼리를 개선하였습니다. (항상 필요한 정보를 지연 로딩 대신 즉시 로딩 + fetch 조인 적용)

📄 작업 내용

@EntityGraph를 이용해 fetch join으로 조회하도록 합니다.

🙋🏻 주의 사항

3인 페어로 진행하여, 준팍 확인하시면 머지 진행하도록 하겠습니다!

(노션의 BE 공간 - 쿼리 개선 기록 참조)

스크린샷

📎 관련 이슈

closed #494

레퍼런스

yoondgu and others added 2 commits September 26, 2023 16:12
Co-authored-by: cpot5620 <cpot5620@gmail.com>
Co-authored-by: kpeel5839 <89840550+kpeel5839@users.noreply.github.com>
Co-authored-by: cpot5620 <cpot5620@gmail.com>
Co-authored-by: kpeel5839 <89840550+kpeel5839@users.noreply.github.com>
@yoondgu yoondgu added BE 백엔드 관련 이슈 우선순위 : 상 refactor 리팩토링 관련 이슈 labels Sep 27, 2023
@yoondgu yoondgu self-assigned this Sep 27, 2023
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.

수고하셨습니다 👍👍👍

@github-actions
Copy link

Unit Test Results

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

Results for commit 3eed629.

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.

우리 고생했어요~~~

@kpeel5839 kpeel5839 merged commit bbde16d into develop-BE-2 Sep 27, 2023
3 checks passed
@yoondgu yoondgu added this to the 최종 데모데이 milestone Oct 3, 2023
@yoondgu yoondgu deleted the refactor/#494-re branch October 6, 2023 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 상
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants