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

[feat] 주문 시점에 Cart에 낙관락을 추가한 버전 #146

Conversation

Hyeon-Uk
Copy link
Contributor

💡 다음 이슈를 해결했어요.

Issue Link - #118

  • CartEntity에 낙관락을 추가하여 문제를 해결한 버전

💡 이슈를 처리하면서 추가된 코드가 있어요.

  • (없다면 이 문항을 지워주세요.)

💡 이런 고민을 했어요.

  • (없다면 이 문항을 지워주세요.)

💡 다음 자료를 참고하면 좋아요.

  • (없다면 이 문항을 지워주세요.)

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

Dr-KoKo added 11 commits August 24, 2024 19:28
낙관락이 걸린 엔티티의 조회는 트랜젝션 내에서 수행되어야 한다.
카트에 상품 추가 요청이 동시적으로 발생하면 재시도하는 로직 추가
재시도는 3번까지 허용
낙관적 락의 획득에 실패해서
1)카트에 메뉴를 못담는 경우 -> RapidAddCartException
2)결제에 실패한 경우 -> DuplicateOrderException
…_결제시_동일한_카트에_대한_중복_결제_문제' into feature/118_Dr-KoKo_Hyeon-Uk_고도화_Optimistic_Lock

# Conflicts:
#	build.gradle
#	src/main/java/camp/woowak/lab/cart/persistence/jpa/entity/CartEntity.java
…_Hyeon-Uk_고도화_결제시_동일한_카트에_대한_중복_결제_문제' into feature/118_Dr-KoKo_Hyeon-Uk_고도화_Optimistic_Lock
@Hyeon-Uk Hyeon-Uk added ✨ Feature 기능 개발 💪 Improve 기능 고도화 & 개선 labels Aug 26, 2024
@Hyeon-Uk Hyeon-Uk added this to the 1차 고도화 milestone Aug 26, 2024
@Hyeon-Uk Hyeon-Uk assigned Hyeon-Uk and unassigned Hyeon-Uk Aug 26, 2024
@Hyeon-Uk Hyeon-Uk requested a review from Dr-KoKo August 26, 2024 12:21
@Hyeon-Uk Hyeon-Uk changed the title [feat] CartEntity에 낙관락을 추가한 버전 [feat] 주문 시점에 Cart에 낙관락을 추가한 버전 Aug 26, 2024
@Dr-KoKo Dr-KoKo closed this Aug 26, 2024
@kimhyun5u kimhyun5u deleted the feature/118_Dr-KoKo_Hyeon-Uk_고도화_Optimistic_Lock branch September 19, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 💪 Improve 기능 고도화 & 개선
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants