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

feat: redis 새로운 방식으로 작성 #163

Merged
merged 7 commits into from
Jan 5, 2025
Merged

feat: redis 새로운 방식으로 작성 #163

merged 7 commits into from
Jan 5, 2025

Conversation

mirageoasis
Copy link
Contributor

⛏ 작업 사항

  • 기존의 락을 사용해서 자리 상태 동시성을 나타내는 상황을 레디스에 저장함으로써 락을 사용하지 않고 동시성 문제를 예방했습니다.
  • 또한, 자리 예약을 하지 않을시 스프링 이벤트를 발행하여 자리 예약을 해지하는 코드를 레디스의 TTL로 대체하였습니다.

📝 작업 요약

  • 레디스를 사용하여 동시성 해결
  • 좌석 점유 해지 이벤트를 코드를 사용하지 않고 레디스를 사용하여 더 간단히 해결.

@mirageoasis
Copy link
Contributor Author

@lass9436 @seminchoi 두분 다 실험 감사합니다! approve 해주시면 머지할께요!

@lass9436
Copy link
Contributor

lass9436 commented Jan 4, 2025

고생하셨습니다!

@mirageoasis mirageoasis merged commit 9e72879 into main Jan 5, 2025
1 check passed
@mirageoasis mirageoasis deleted the feat/new-redis-reservation branch January 5, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hseong3243 hseong3243 hseong3243 approved these changes

@lass9436 lass9436 lass9436 approved these changes

@seminchoi seminchoi Awaiting requested review from seminchoi seminchoi is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mirageoasis @lass9436 @hseong3243