Skip to content

Conversation

@leebs0521
Copy link
Collaborator

resolved :

📌 과제 설명

jpa open in view를 false로 변경

image

open-in-view에 관한 이야기

👩‍💻 요구 사항과 구현 내용

✅ PR 포인트 & 궁금한 점

@leebs0521 leebs0521 self-assigned this Jan 14, 2025
@leebs0521 leebs0521 linked an issue Jan 14, 2025 that may be closed by this pull request
1 task
@m-a-king
Copy link
Collaborator

잘 읽어봤습니다. 저희는 lazy loading을 사용하지 않으니 문제가 없겠군요. 트랜잭션은 파면 팔수록 어려운 것 같아요

@leebs0521
Copy link
Collaborator Author

@m-a-king @lob이 지연로딩 방식이라고하는데 저희는 필요시 @Trascational 안에서 DTO로 반환해서 @lob 붙은 컬럼 접근해서 상관없을것 같습니다.

@m-a-king
Copy link
Collaborator

@m-a-king @lob이 지연로딩 방식이라고하는데 저희는 필요시 @Trascational 안에서 DTO로 반환해서 @lob 붙은 컬럼 접근해서 상관없을것 같습니다.

괜찮다고 하셨는데 괜찮지 않으셔서 변경하시고 이후 PR 올려주신 건가요?

@leebs0521
Copy link
Collaborator Author

@m-a-king
open-in-view 가 true면 API 호출시부터 응답까지 트랜잭션이 유지된다고 합니다.
근데 false면 @Transtional이 붙은 메서드 호출 동안만 트랙잰션이 유지되서 상관 없다는 이야기 였어요.

예를들어
모집글 상세조회 API 호출시
RecruitQueryService.getRecruitBoardById(id)가 호출되서 ResponseDto가 반환되니까
RecruitQueryService.getById(id)는 @Trasctional이 붙어있어서 내부에서 @lob이 붙은 content가 지연로딩 되도 문제 없다는 이야기였습니다.

image

@m-a-king
Copy link
Collaborator

@m-a-king open-in-view 가 true면 API 호출시부터 응답까지 트랜잭션이 유지된다고 합니다. 근데 false면 @Transtional이 붙은 메서드 호출 동안만 트랙잰션이 유지되서 상관 없다는 이야기 였어요.

예를들어 모집글 상세조회 API 호출시 RecruitQueryService.getRecruitBoardById(id)가 호출되서 ResponseDto가 반환되니까 RecruitQueryService.getById(id)는 @Trasctional이 붙어있어서 내부에서 @lob이 붙은 content가 지연로딩 되도 문제 없다는 이야기였습니다.

image

이해했습니다. lob이 지연 로딩이라고 해도 저희는 트랜잭션 내부에서 모든 로직을 처리하니 문제가 없다는거군요~ 사실 너무 당연한 느낌이네요. 왜 true 설정이 기본값인지...

@leebs0521 leebs0521 force-pushed the fix/298-set-jpa-open-in-view-false branch from 255378e to 98b63e9 Compare January 15, 2025 03:25
 - test yml 파일에도 추가
@sonarqubecloud
Copy link

@leebs0521 leebs0521 merged commit 01cdddf into main Jan 15, 2025
3 checks passed
@leebs0521 leebs0521 deleted the fix/298-set-jpa-open-in-view-false branch January 15, 2025 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] jpa open in view

4 participants