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

test: 분산락 테스트 작성 #101

Merged
merged 8 commits into from
Nov 26, 2024
Merged

Conversation

kimyu0218
Copy link
Collaborator

Issue Number

close: #

작업 개요

Redisson 분산락 테스트 작성

작업 사항

  • 분산락의 문제는 아니었지만 여전히 미션 인증 관련 동시성 문제가 발생하여 분산락 테스트를 작성했습니다.
  • 최근 슬랙 dev 채널에 미션 삭제 예고 알림 이슈가 올라왔습니다. 당시에는 검증 로직이 부족하여 발생한 문제라고 생각했는데 검증 로직이 올바르게 작성되어 있어 원인 파악을 위해 로그를 추가했습니다 😢😢
  • Mission 도메인에 작성한 주석을 JavaDoc으로 바꿨습니다..!

고민한 점들(필수 X)

여기에 작성하세요

스크린샷(필수 X)

여기에 작성하세요

@kimyu0218 kimyu0218 requested a review from songyi00 as a code owner November 24, 2024 15:51
Copy link
Member

@songyi00 songyi00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생했습니당 👍

@kimyu0218 kimyu0218 merged commit 10f1a00 into develop Nov 26, 2024
1 check passed
@kimyu0218 kimyu0218 deleted the test/#96-distributed-lock branch November 26, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants