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.

Sign up for GitHub

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

Jump to bottom

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

Closed
1 task done
hoonyworld opened this issue Oct 5, 2024 · 0 comments · Fixed by #230
Closed
1 task done

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

hoonyworld opened this issue Oct 5, 2024 · 0 comments · Fixed by #230
Assignees
@hoonyworld
Labels
bug Something isn't working 동훈🐻

Comments

@hoonyworld
Copy link
Member

hoonyworld commented Oct 5, 2024 •
edited
Loading

Bug/Fix Issue 📌

image
  • handleSocialLogin 메서드에서 트랜잭션이 이미 시작되었기 때문에, registerMemberWithUserInfo 메서드는 기존 트랜잭션에 참여하게 됩니다.
  • memberRegistrationService.registerMemberWithUserInfo(memberInfoResponse)를 통해 새로운 회원을 등록할 때, 해당 데이터는 트랜잭션이 커밋되기 전까지 데이터베이스에 즉시 반영되지 않습니다.
  • 따라서 이후에 memberService.findUserByMemberId(memberId)를 호출하면, 방금 등록한 회원이 데이터베이스에 아직 반영되지 않았기 때문에 NotFoundException이 발생하게 된다고 생각이 듭니다.
  • 따라서 flush()를 호출하여 트랜잭션을 중간에 커밋하지 않고도 데이터베이스와 동기화하고자 합니다.

Todo ✔️

  • flush()를 호출하여 트랜잭션을 중간에 커밋하지 않고도 데이터베이스와 동기화
@hoonyworld hoonyworld added bug Something isn't working 동훈🐻 labels Oct 5, 2024
@hoonyworld hoonyworld self-assigned this Oct 5, 2024
hoonyworld added a commit that referenced this issue Oct 5, 2024
@hoonyworld
[#229] fix(SocialLoginService): 회원 등록 후 flush()를 호출하여 DB에 반영
8d4571c
@hoonyworld hoonyworld mentioned this issue Oct 5, 2024
Merged
@hoonyworld hoonyworld linked a pull request Oct 5, 2024 that will close this issue
[HOTFIX] #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 #230
Merged
hoonyworld added a commit that referenced this issue Oct 5, 2024
@hoonyworld
[#229] refactor: flush 호출 위치 변경
a7ae443
hoonyworld added a commit that referenced this issue Oct 5, 2024
@hoonyworld
HOTFIX #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 (#230)
f76896a 
* [#229] fix(SocialLoginService): 회원 등록 후 flush()를 호출하여 DB에 반영

* [#229] refactor: flush 호출 위치 변경
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoonyworld hoonyworld

Labels
bug Something isn't working 동훈🐻
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[HOTFIX] #229 - 회원 등록 후 flush()를 호출하여 DB에 반영 TEAM-BEAT/BEAT-SERVER
1 participant
@hoonyworld