Skip to content

[refactor/#694] MemberExercise 참조 방향 및 도메인 네이밍 정리 - #747

Merged
Dimo-2562 merged 6 commits into
developfrom
refactor/#694
Sep 16, 2026
Merged

Dimo-2562 merged 6 commits into
developfrom
refactor/#694

Conversation

@Dimo-2562

Copy link
Copy Markdown
Contributor

❤️ 기능 설명

MemberExercise를 ExerciseParticipation으로 변경하고 운동 참여의 소유권과 삭제 책임을 exercise 도메인 기준으로 정리했습니다.
기존 API와 member_exercise 테이블은 유지합니다.

변경 내용

  • 참여 엔티티·repository·reader 및 관련 참조 명칭 통일
  • Member의 운동 참여 역방향 연관관계 제거
  • soft delete와 hard delete용 참여 정리 흐름 분리
  • 회원·운동 삭제 생명주기 회귀 테스트 추가

연결된 issue

close #694



🩷 Approve 하기 전 확인해주세요!

  • 향후 회원 hard delete 구현 시 ExerciseParticipationCleanupService.prepareMemberHardDelete(memberId)를 Member 삭제 전에 같은 삭제 흐름에서 호출해야 합니다.
  • public API와 DB member_exercise 테이블은 변경하지 않았습니다.

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가?
  • 이슈넘버를 적었는가?

@Dimo-2562
Dimo-2562 requested a review from kanghana1 September 13, 2026 01:41
@Dimo-2562 Dimo-2562 self-assigned this Sep 13, 2026
@Dimo-2562 Dimo-2562 added the 🌟 REFACT New feature or request label Sep 13, 2026

@kanghana1 kanghana1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

고생하셨습니다! 추가로 중요한 건 아니지만, MemberExercise를 ExerciseParticipation으로 리네이밍 하셨는데 에러코드는 MEMBER_EXERCISE_NOT_FOUND 이거 그대로인 거 같아서 통일해주시면 좋을 거 같습니다!!

@Param("exerciseId") Long exerciseId);

@Modifying
@Modifying(flushAutomatically = true)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Modifying 벌크 삭제에 flushAutomatically만 있고 clearAutomatically가 없어, 같은 트랜잭션에서 이미 로딩된 ExerciseParticipation/exercise.participations를 이후 재조회하면 1차 캐시의 stale 값이 반환될 수 있을 거 같습니다!

@Dimo-2562

Dimo-2562 commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor Author

고생하셨습니다! 추가로 중요한 건 아니지만, MemberExercise를 ExerciseParticipation으로 리네이밍 하셨는데 에러코드는 MEMBER_EXERCISE_NOT_FOUND 이거 그대로인 거 같아서 통일해주시면 좋을 거 같습니다!!

a43b9c0 에서 반영하였습니다!

@Dimo-2562
Dimo-2562 merged commit f2e0013 into develop Sep 16, 2026
1 check passed
@Dimo-2562
Dimo-2562 deleted the refactor/#694 branch September 16, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌟 REFACT New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACT] MemberExercise 참조 방향 및 도메인 네이밍 정리

2 participants