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

refactor: 프로젝트 리스트 조회 최적화 #171

Merged
merged 1 commit into from
Sep 8, 2024

Conversation

Ares-10
Copy link
Contributor

@Ares-10 Ares-10 commented Sep 8, 2024

📍 PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #170

🔁 변경 및 추가사항

  1. 이전에 사용했던 fetchResult 함수는 더이상 사용되지 않기에 fetch, fetchOne 함수로 교체해주었습니다.
  2. PageableExecutionUtils.getPage를 사용해 count 쿼리가 필요할때만 발생하도록 했습니다.
  3. leftJoin(project.imageList, image).fetchJoin()을 통해 n+1 문제를 해결했습니다.

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@Ares-10 Ares-10 added the ♻️ Refactor 코드 리팩토링 label Sep 8, 2024
@Ares-10 Ares-10 self-assigned this Sep 8, 2024
@Ares-10 Ares-10 merged commit e9fc9ca into develop Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 프로젝트 목록 조회 최적화
2 participants