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] #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 #230

Merged
merged 2 commits into from
Oct 5, 2024

Conversation

hoonyworld
Copy link
Member

@hoonyworld hoonyworld commented Oct 5, 2024

Related issue 🛠

Work Description ✏️

flush() 호출

  • role을 Member로 등록하고 저장 시, flush()를 호출하여 변경 사항을 데이터베이스에 반영하도록 했습니다.

  • 회원 정보 조회 시, memberService.findUserByMemberId(memberId)를 호출하여 방금 등록한 회원의 Users 정보를 조회할 때, 데이터베이스에 변경 사항이 반영되어 있으므로, 정상적으로 데이터를 가져올 수 있게 됩니다.

Propagation.REQUIRES_NEW

  • 대안으로 Propagation.REQUIRES_NEW를 이용해 새로운 트랜잭션을 시작하여 회원 등록을 커밋할 수 있지만, 상위 트랜잭션이 롤백되더라도 하위 트랜잭션의 변경 사항은 롤백되지 않아 데이터 일관성 문제가 발생할 수 있다고 해서 사용하지 않았습니다.

Trouble Shooting ⚽️

Related ScreenShot 📷

Uncompleted Tasks 😅

To Reviewers 📢

트랜잭션 전파 관련(ROLE을 USER로 부여하는 문제)

  • 제 생각에는 트랜잭션 전파 관련 역할을 MEMBER로 저장하는 과정이 flush되지 않고, 후속 로직이 실행되기 때문에 역할이 default값인 USER로 들어갔을 수도 있다고 판단됩니다!!

@hoonyworld hoonyworld merged commit f76896a into develop Oct 5, 2024
1 check passed
@hoonyworld hoonyworld deleted the bug/#229 branch October 5, 2024 18:50
@hoonyworld hoonyworld changed the title HOTFIX #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 [HOTFIX] #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 Oct 6, 2024
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.

[bug] 회원가입 시 트랜잭션 경계 문제
1 participant