fix: 같은 회원의 동시 예약이 하루 3시간 한도를 넘지 않게 한다

[ff1451]

요약

하루 이용 한도(3시간) 검사가 그날 회원의 기존 예약 행만 잠갔다(findActiveByMemberIdAndDateForUpdate). 기존 예약이 없는 날에는 잠글 행이 없어, 같은 회원이 겹치지 않는 2시간 예약 두 건을 동시에 보내면 둘 다 통과해 4시간이 예약됐다.

변경

• 예약 생성 시작 시 회원 행을 잠가(memberRepository.findByIdForUpdate) 한 회원의 예약 생성을 직렬화한다. 생성 트랜잭션은 외부 호출 없이 짧다.
• 반복 예약 저장 중 다른 회원과 동시에 겹쳐 DB 배타 제약에 걸리면, 단건 예약과 같이 409 OVERLAPPING_RESERVATION으로 응답한다(이전: 전역 처리기의 일반 400). JPA에서는 한 건만 되돌리는 세이브포인트를 쓸 수 없어 전체 롤백은 이전과 같다 — 다시 요청하면 겹친 날짜는 건너뛰고 생성된다.

테스트

• ReservationConcurrencyTest에 추가: 같은 회원이 빈 날에 10:00–12:00, 13:00–15:00을 동시에 요청하면 하나는 200, 하나는 409, 예약은 1건(5일 반복).
  • 수정 전 코드에서는 [200, 200]으로 실패함을 확인했다.
  • 수정 후 클래스 전체를 8번 연속 돌려 통과.
• 로컬 전체 ./gradlew test: 550건 중 549 통과, 1 스킵.

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 42 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 78026b68-be0f-495f-8864-338b51423355

📥 Commits

Reviewing files that changed from the base of the PR and between b3af697 and 98fbdf5.

📒 Files selected for processing: 2

• src/main/java/com/bcsdlab/bcsdinternalapiv2/reservation/service/ReservationService.java
• src/test/java/com/bcsdlab/bcsdinternalapiv2/reservation/ReservationConcurrencyTest.java

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

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