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

[HIMIN-236] fix : finishDelivery 트랜잭션 분리 -> 결합 #120

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

BeommoKoo-dev
Copy link
Collaborator

PR 확인 항목

PR 보내기전에 아래 항목들을 만족 하였는지 체크 해주세요

  • 곤모슬 팀에서 정한 커밋 메시지 규칙과 일치하는가?
  • 곤모슬 팀에서 정한 코딩 컨벤션과 일치하는가?

PR 종류

어떤 종류의 PR인지 아래 항목중에 체크 해주세요

  • 버그 수정
  • 기능 추가
  • 코드 스타일 변경 (formatting, local variables)
  • 리팩토링 (기능 변경 X)
  • 빌드 관련 수정
  • 문서 내용 수정
  • 테스트 추가
  • 그 외, 어떤 종류인지 기입 바람:

어떤 기능이 추가 되었나요?

Issue Number: HIMIN-236

기존에 있던 기능에 영향을 주나요?

  • 아니요

기타

기존에는 Delivery 도메인의 배달 종료 히스토리 저장 이후 트랜잭션을 분리하여 이벤트 처리로 Order에게 배달 완료를 전달했는데,
트랜잭션을 분리할 시 생길 수 있는 문제점이, Order의 배달종료 히스토리 저장에 실패할 경우 Order에는 배달 완료 히스토리가 저장이 되지 않지만,
Delivery 도메인의 배달 종료 히스토리 저장은 다른 트랜잭션이라, Delivery의 배달 종료는 정상 저장되어 데이터의 정합성이 깨지는 문제를 발견하여 Delivery의 배달종료 히스토리 저장 - Order의 배달완료 히스토리 저장을 같은 트랜잭션으로 묶어주었습니다.

@BeommoKoo-dev BeommoKoo-dev merged commit 0eb3d52 into main Sep 19, 2023
1 check passed
@BeommoKoo-dev BeommoKoo-dev deleted the HIMIN-236-beommo-finishdevliery-refactor branch September 19, 2023 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants