Skip to content

Conversation

@Chuseok22
Copy link
Member

@Chuseok22 Chuseok22 commented Jan 5, 2026

…m//issues/701

✨ 변경 사항


✅ 테스트


  • 수동 테스트 완료
  • 테스트 코드 완료

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 콘서트 조회 쿼리 최적화: 불필요한 필드 제거 및 데이터 구조 간소화
    • 내부 필드 별칭 정리로 쿼리 성능 개선

✏️ Tip: You can customize this high-level summary in your review settings.

@Chuseok22 Chuseok22 self-assigned this Jan 5, 2026
@Chuseok22 Chuseok22 added the fix 버그 수정 (Fix) label Jan 5, 2026
@Chuseok22 Chuseok22 merged commit 8785002 into test Jan 5, 2026
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Jan 5, 2026

Caution

Review failed

The pull request is closed.

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'ignored_branch'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

워크스루

ConcertRepositoryImpl의 filteredConcert 및 filteredConcertForAdmin 쿼리에서 createdDate 필드를 선택 및 GROUP BY 절에서 제거하고, ticketPreOpenDate, ticketGeneralOpenDate, ticketOpenDate 등의 필드 별칭을 제거하여 공개되는 별칭 수를 감소시키고 쿼리 결과를 단순화했습니다.

변경 사항

응집 / 파일 변경 요약
쿼리 필드 및 별칭 정리
ConcertRepositoryImpl.java
createdDate를 쿼리 선택 및 GROUP BY에서 제거; ticketPreOpenDate, ticketGeneralOpenDate, ticketOpenDate 별칭 제거하여 불필요한 필드 별칭 감소

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~12분

관련 가능 PR

  • PR #702: 동일한 파일의 GROUP BY 및 별칭 동작을 변경하며, createdDate 제거 및 별칭 제거라는 동일한 주제를 다룸
  • PR #406: ConcertRepositoryImpl의 filteredConcert/filteredConcertForAdmin 쿼리 및 Q-entity 사용을 함께 수정
  • PR #237: ticketPreOpenDate/ticketGeneralOpenDate 필드 및 관련 GROUP BY 로직에 대한 쿼리 프로젝션 수정

🐰 별칭들을 정리하고 createdDate를 빼니,
쿼리 결과는 한결 깔끔해졌군요!
불필요한 필드는 사라지고,
핵심만 남겨서 빛이 나네요,
여름 바람처럼 시원한 리팩토링! ✨

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cfa57cd and 51581d0.

📒 Files selected for processing (1)
  • ticketmate-concert/src/main/java/com/ticketmate/backend/concert/infrastructure/repository/ConcertRepositoryImpl.java

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 and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정 (Fix)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

❗ [버그][공연] 공연 필터링 조건 최신순/오픈일순 변경 시 응답값 동일

2 participants