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

[FIX] UserMission 업데이트 쿼리에서의 비관적 락 문제 해결 #134

Merged
merged 4 commits into from
Mar 6, 2024
Merged

[FIX] UserMission 업데이트 쿼리에서의 비관적 락 문제 해결 #134

merged 4 commits into from
Mar 6, 2024

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Mar 6, 2024

📌 관련 이슈

closed #133

✨ 어떤 이유로 변경된 내용인지

  • UserMission의 mission_id와 completed_status 를 업데이트 하는 jpql 쿼리에서 동시성 문제에 의한 비관적 락이 걸리는 문제를 해결하고자 함
  • 업데이트 쿼리 호출 횟수에 따라 version을 명시하거나, 비관적 락 -> 낙관적 락으로 변경하는 등의 방안을 떠올렸으나, 현재는 객체 그래프 탐색으로 대체해둔 상태 *추후 논의점
  • 온보딩 매칭 API 에서 초대를 수신하는 측의 필터링 조건 추가

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

@jun02160 jun02160 added 🚨HOTFIX 급한 수정사항 발생 🐞BUGFIX 에러 수정, 버그 수정 🔥PULL REQUEST PR 날릴 때 🧹MODIFY 코드 수정 (기능의 변화가 있을 때) 🐈예준🐈 labels Mar 6, 2024
@jun02160 jun02160 self-assigned this Mar 6, 2024
@jun02160 jun02160 merged commit ac04342 into develop Mar 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jun02160 jun02160

Labels
🐈예준🐈 🐞BUGFIX 에러 수정, 버그 수정 🚨HOTFIX 급한 수정사항 발생 🧹MODIFY 코드 수정 (기능의 변화가 있을 때) 🔥PULL REQUEST PR 날릴 때
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FIX] 오늘의 미션 조회 버그 해결
1 participant
@jun02160