Skip to content

hotfix: @Where 어노테이션 임시 수정 #239

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.

Sign up for GitHub

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

Merged
yongjun0511 merged 1 commit into from
Sep 22, 2025
Merged

hotfix: @Where 어노테이션 임시 수정 #239

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/main/java/ssu/eatssu/domain/partnership/entity/Partnership.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
@NoArgsConstructor(access = AccessLevel.PROTECTED)
@Builder
@AllArgsConstructor
@Where(clause = "start_date <= CURRENT_DATE AND end_date >= CURRENT_DATE")
@Where(clause = "end_date >= CURRENT_DATE")
Copy link

@gemini-code-assist gemini-code-assist bot Sep 22, 2025

Choose a reason for hiding this comment

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

high

이 변경으로 인해 아직 시작되지 않은 제휴도 조회 결과에 포함되게 됩니다. 이는 의도된 동작으로 보입니다.

다만 @Where 어노테이션은 해당 엔티티에 대한 모든 조회 쿼리에 전역적으로 필터 조건을 추가하므로, 의도치 않은 부작용을 일으킬 수 있습니다. 예를 들어, 관리자가 만료된 제휴 정보를 조회하거나 수정해야 할 경우, @Where 조건 때문에 엔티티를 불러오지 못하는 문제가 발생할 수 있습니다.

장기적으로는 @Where를 사용하는 대신, 다음과 같은 방법을 고려해 보시는 것을 추천합니다:

  • 각각의 조회 요구사항에 맞는 별도의 리포지토리 메소드를 작성 (예: findActivePartnerships, findAllIncludingFuturePartnerships)
  • JPA Specification이나 QueryDSL을 사용하여 동적으로 쿼리 조건을 조합

추가적으로, 이 변경으로 인해 PartnershipRepository의 findRestaurantsWithMyPartnerships 쿼리에 있는 p.endDate >= current_date 조건이 중복됩니다. @Where가 적용되므로 해당 쿼리 내의 조건은 제거해도 동일하게 동작할 것입니다. 이는 @Where가 코드베이스 전반에 미치는 영향을 보여주는 좋은 예시입니다.

이 수정이 임시 조치임을 감안할 때, 추후 리팩토링을 위한 이슈를 생성해두는 것도 좋은 방법입니다.

public class Partnership {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
Expand Down