Skip to content

Feat/205 정원 축소 시 자동 강등#211

Closed
jaylovegood wants to merge 34 commits into204-신청취소-가능-시점-수정from
feat/205-정원-축소-시-자동-강등

Hidden character warning

The head ref may contain hidden characters: "feat/205-\uc815\uc6d0-\ucd95\uc18c-\uc2dc-\uc790\ub3d9-\uac15\ub4f1"
Closed

Feat/205 정원 축소 시 자동 강등#211
jaylovegood wants to merge 34 commits into204-신청취소-가능-시점-수정from
feat/205-정원-축소-시-자동-강등

Conversation

@jaylovegood
Copy link
Collaborator

@jaylovegood jaylovegood commented Mar 15, 2026

✨ Feature PR (to dev)

중요: 202 -> 203 -> 204 -> 205 순으로 리뷰, 머지해주세요!!!

🧪 로컬 테스트 여부 (작업자 체크)

  • 로컬에서 Swagger 혹은 테스트 코드로 동작을 확인했습니다.

📄 documentation 최신화 여부

변경사항과 관련된 API의 swagger documentation이 실제 동작과 일치하는지 확인합니다.

  • (작업자) 확인하였습니다.
  • (리뷰어) 확인하였습니다.

📌 작업 내용(what & why)

  • 정원 축소를 허용하며, 이로 인해 정원 밖으로 밀려나는 후순위 참여자는 대기자로 강등하는 로직 추가
  • 대기자 강등 시 메일 발송
  • 정원 축소 차단을 위해 쓰이던 에러코드 제거
  • 이슈와는 무관하나 기존에 있던 이메일 제목들 조금씩 수정

🔎 영향 범위

  • 동시성 / 트랜잭션 처리 영향

👀 집중 리뷰 요청사항 (있다면)

  • 남아있는 동시성 이슈가 없는지 다시 한 번 검토하면 좋을 것 같습니다.

🔥 관련 이슈

@jaylovegood jaylovegood requested a review from uykhc March 15, 2026 12:52
@uykhc
Copy link
Collaborator

uykhc commented Mar 16, 2026

  • 여기도 마찬가지로 데드락 가능성이 있다고 합니다.
    정원 축소 강등 로직이 기존 참가자 취소 로직과 맞물립니다.
    EventService.update()는 이벤트 락을 잡은 뒤 강등 대상 registrations를 saveAll로 갱신하고, 반대로 참가자 취소는 registration 삭제 후 reconcileWaitlist()에서 이벤트 락을 잡습니다.(이 리뷰를 작성하는 시점에 지수님이 수정해주셨을 수도 있을 것 같습니다.)

현재는 waitlistEnabled=false여도 정원 축소 시 강제로 WAITLISTED로 강등된다고 합니다.

Copy link
Collaborator

@uykhc uykhc left a comment

Choose a reason for hiding this comment

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

확인했습니다. 수고하셨습니다!

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.

2 participants