태그 동시 생성 시 중복 문제 해결 #978
Merged
태그 동시 생성 시 중복 문제 해결 #978
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kyum-q
requested changes
Dec 19, 2024
backend/src/main/java/codezap/template/service/facade/TemplateApplicationService.java
Show resolved
Hide resolved
backend/src/test/java/codezap/template/service/facade/TemplateApplicationServiceTest.java
Outdated
Show resolved
Hide resolved
jminkkk
force-pushed
the
fix/960-resolve-duplicate-tag
branch
from
c6cbdb4 to
448e036
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
⚡️ 관련 이슈
close #960
📍주요 변경 사항
고민해본 방법
위와 같았어요. 각 이유에 대해 우려되는 점들은 이슈에 작성해놓았습니다!
결론적으로는 유니크 제약 조건과 재시도로 처리하는 게 현재 상황에서 적절하다고 판단했어요!
따라서 재시도를 처리해주는 spring retry 의존성을 추가했습니다.
기존 정책에 의한 명시적 제약 조건 추가
JPA에서 제공하는 unique 제약 조건은 대소문자를 구분하지 않아요.
때문에 기존 테스트 코드 중 "대소문자 구분하여 조회하는지"에 대한 테스트가 실패했어요.
우리의 정책은 태그의 대소문자를 구분하기 때문에 columnDefinition 옵션에 제약을 추가했으니 참고 부탁드립니다!
🎸 기타
문제 상황을 재연하기 위한 테스트에서 ServiceTest에 @transactional 어노테이션으로 인해 재연이 불가능해요.
@disable 처리를 해놓았으니 문제 상황을 확인하고 싶으신 분들은 참고 부탁해요!
🍗 PR 첫 리뷰 마감 기한
12/20 13:00