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

hotfix: OSIV false 설정으로 인한 스케줄링 메서드 트랜잭션 선언 #617

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

hyeon0208
Copy link
Contributor

🚩 연관 이슈

close #614


📝 작업 내용

image

OSIV false 설정으로 인해 해당 메서드까지 영속성 컨텍스트가 살아있지 못해서 Lazy 예외가 발생
=> 트랜잭션 선언으로 영속성 컨텍스트 생명 범위 확장


🏞️ 스크린샷 (선택)


🗣️ 리뷰 요구사항 (선택)

Copy link

Test Results

148 tests  ±0   148 ✅ ±0   5s ⏱️ ±0s
 43 suites ±0     0 💤 ±0 
 43 files   ±0     0 ❌ ±0 

Results for commit a6c7ad4. ± Comparison against base commit 651acc0.

Copy link

📝 Test Coverage Report

Overall Project 79.66%
Files changed 100% 🍏

File Coverage
NotificationService.java 98.28% 🍏

@hyeon0208 hyeon0208 changed the title fix: OSIV false 설정으로 인한 스케줄링 메서드 트랜잭션 선언 hotfix: OSIV false 설정으로 인한 스케줄링 메서드 트랜잭션 선언 Sep 26, 2024
@hyeon0208 hyeon0208 merged commit 3321e96 into main Sep 26, 2024
3 checks passed
@hyeon0208 hyeon0208 deleted the hotfix/614 branch September 26, 2024 14:26
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.

4 participants