Skip to content
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.

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

feat: delete jpql 에 @Transactional 추가 #714

Closed
wants to merge 1 commit into from

Conversation

yeon-06
Copy link
Collaborator

@yeon-06 yeon-06 commented Nov 26, 2022

요약



작업 내용

delete 하는 JPQL에 @Transactional 추가



참고 사항

  • 해당 어노테이션을 붙인 이유에 대해서는 이슈에 정리해두었습니다.

  • 해당 메서드들에도 @Transactional을 붙일지 고민됩니다.

    • 고민하는 이유: 이번 리팩터링의 목적이 트랜잭션 없는 환경에서 Repository를 호출해도 예외가 발생하지 않게 하기 위함인데 조회 JPQL은 트랜잭션이 없어도 동작은 됩니다. 다만 트랜잭션이 없는 상태에서 호출하는 경우 영속성 관련 이슈가 발생할 수 있는데 조회만 하는 상황에서 영속성을 관리해야할까? 라는 의문이 들기도 하고 고민되네요🤔



관련 이슈



@yeon-06 yeon-06 added 🎉 BE 백엔드 관련 ⚙️ REFACTOR labels Nov 26, 2022
@yeon-06 yeon-06 self-assigned this Nov 26, 2022
@github-actions
Copy link

Unit Test Results

235 tests   235 ✔️  16s ⏱️
  89 suites      0 💤
  89 files        0

Results for commit 338b49e.

@pickpick-sonarqube
Copy link

Passed

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage No coverage information (87.60% Estimated after merge)
  • Duplications 0.00% Duplicated Code (0.00% Estimated after merge)

Project ID: woowacourse-teams_2022-pickpick_AYKprLeNXDQxKhlck1fc

View in SonarQube

@yeon-06 yeon-06 closed this Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@Query 사용 시 @Transactional 추가
1 participant