Skip to content

Conversation

@hyeminleeee
Copy link
Contributor

@hyeminleeee hyeminleeee commented Aug 18, 2025

📌 PR 개요

  • 서비스 레이어의 @transactional 어노테이션을 메서드별로 세밀하게 관리하도록 리팩토링
  • 읽기 전용 메서드와 쓰기 작업 메서드를 구분하여 성능 최적화 및 명시성 향상

🔍 변경 사항

  • @transactional 어노테이션 임포트 통일 : Jakarta EE 임포트를 Spring Framework 임포트로 변경

  • MatchService: 클래스 레벨 @transactional 제거 후 메서드별 적용

  • MatchParticipantService: 클래스 레벨 @transactional(readOnly = true) 제거

  • UserService: 메서드별 트랜잭션 타입 최적화

🧪 테스트 방법

1.경기 목록 조회 API 호출하여 정상 응답 확인
2.경기 생성, 참여 신청, 참여 취소, 삭제 기능 정상 동작 확인
3.내 참여 경기 조회 기능 정상 동작 확인
4.사용자 조회 및 생성/업데이트 기능 정상 동작 확인

✅ 체크리스트

  • 코드가 정상적으로 동작함
  • 기존 기능에 문제가 없음
  • 코드 컨벤션(Prettier/ESLint, Checkstyle 등)을 통과함
  • 필요 시 문서 업데이트 완료
  • 관련 이슈에 연결함

📎 참고 이슈

Ref: #79

@hyeminleeee hyeminleeee requested a review from a team August 18, 2025 01:31
Mminzy22
Mminzy22 previously approved these changes Aug 18, 2025
@Mminzy22 Mminzy22 merged commit 6c82a2e into develop Aug 18, 2025
1 check passed
@Mminzy22 Mminzy22 deleted the feature/79-transactional-update branch August 18, 2025 01:41
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.

3 participants