Skip to content

[#35] 경매 목록 조회 성능 개선 - #36

Merged
JIWON27 merged 7 commits into
mainfrom
refactor/35
Sep 29, 2026
Merged

JIWON27 merged 7 commits into
mainfrom
refactor/35

Conversation

@JIWON27

@JIWON27 JIWON27 commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

개요

  • 경매 목록 조회 API에서 발생하던 N+1 문제 해결
    • Auction → Camera → User(owner) 연관관계를 fetch join으로 즉시 로딩하도록 변경
    • 썸네일 이미지도 카메라 건당 개별 조회하던 방식에서 findByCameraIdIn으로 한 번에 조회 후 Map으로 매핑하는 방식으로 변경
  • 위 조회 쿼리의 필터(status, extended_closes_at)/정렬(created_at DESC, id DESC) 조건에 맞춰 auctions, camera_images 테이블에 인덱스 추가

메인 리뷰어 지정 및 Due Date

  • 메인 리뷰어 : blue , Due Date : 2026-09-28

리뷰 시 참고 사항

  • 성능 테스트 관련한 보고서는 노션 링크 공유했습니다. 잘못된 접근법이 있으면 피드백 주시면 감사하겠습니다!
  • auction 테이블 인덱스에 extended_closed_at까지 걸었는데, 이건 입찰 성능 테스트할 때 영향을 줄 수 있다고 생각합니다.

TODO

  • RPS를 더 올릴 수 있는 방법 고민하기

References

[1]

체크리스트

  • PR 제목을 명령형으로 작성했습니다.
  • PR을 연관되는 github issue에 연결했습니다.
  • 리뷰 리퀘스트 전에 셀프 리뷰를 진행했습니다
  • 변경사항에 대한 테스트코드를 추가했습니다. 또는, 테스트 코드가 필요없는 이유가 있습니다.

- AuctionCustomRepositoryImpl: leftJoin+fetchJoin으로 camera/user 즉시 로딩
- CameraImageRepository/AuctionService: 썸네일 조회를 findByCameraIdIn 배치 조회로 변경
- constant_rps rate 10→1, preAllocatedVUs 20→10, maxVUs 100→30
@JIWON27
JIWON27 requested a review from blue000927 September 25, 2026 13:25
@JIWON27 JIWON27 self-assigned this Sep 25, 2026
@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c0cb02e9-6836-4236-8354-f8bde65050b4


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@JIWON27 JIWON27 linked an issue Sep 25, 2026 that may be closed by this pull request
@sonarqubecloud

Copy link
Copy Markdown

@JIWON27
JIWON27 merged commit bce7871 into main Sep 29, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] 경매 목록 조회 성능 개선

1 participant