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

좌석 정보 예약 기능 구현 #11

Open
4 of 5 tasks
hyejungg opened this issue Sep 19, 2021 · 0 comments · Fixed by #15
Open
4 of 5 tasks

좌석 정보 예약 기능 구현 #11

hyejungg opened this issue Sep 19, 2021 · 0 comments · Fixed by #15
Assignees
@hyejungg
Labels
feature 기능 구현
Milestone
예약 관련 기능 구현

Comments

@hyejungg
Copy link
Collaborator

hyejungg commented Sep 19, 2021 •
edited
Loading

  • 공연 좌석 정보 조회하기

  • 좌석 정보 예약하기

    • 동시성 문제 해결하기 (Redis 이용?)

  • 공연 좌석 예약 정보 불러오기

  • 유닛테스트 추가
@hyejungg hyejungg added the feature 기능 구현 label Sep 19, 2021
@hyejungg hyejungg added this to the 예약 관련 기능 구현 milestone Sep 19, 2021
@hyejungg hyejungg self-assigned this Sep 19, 2021
hyejungg added a commit that referenced this issue Oct 2, 2021
@hyejungg
[fix:#11] 피드백 반영
5085387 
- 피드백을 반영하여 getIsBooking()메소드에서 seat 정보를 2번 read 하는 작업을 -> 1개로 수정하였습니다. 
- history에 저장되는 메소드 이름을 saveBookingXXXlog(fail,success)통일 시켰습니다.
- insert, update 등이 수행되는 부분에는 @Transcational을 추가하고, 조회하는 경우에는 readOnly를 추가하였습니다.
hyejungg added a commit that referenced this issue Oct 5, 2021
@hyejungg
[feat:#11] 좌석 예약 동시성 코드 추가, BookingHistory 관련 코드 이동, enum 패키지 생성
46a24a3 
1. enum 패키지를 생성하여 dto에서 분리하였습니다.
2. Redission을 이용하여 분산락 구현하였습니다.
3. SeatService에서 getIsBooking 시 Seat 테이블의 is_booking 여부가 T/F 상관없이 seat_id를 가져오기 위해 관련 메소드를 수정하였습니다.
4. 기존에 예약 실패 시 저장되지 않는 문제를 @Transcational 어노테이션을 제거하여 해결하고, 관련 저장 메소드를 BookingHistoryService로 옮겼습니다.
hyejungg added a commit that referenced this issue Oct 5, 2021
@hyejungg
[fix:#11] redisson wait_time, lease_time 수정
1e159be 
1. BookingService에서  redisson wait_time, lease_time 을 1초, 1초 -> 1초, 2초로 수정하였습니다.
2. interruptedException이 발생할 경우 globalhandler에서 예외를 잡을 수 있도록 runtime에러를 던지도록 수정
hyejungg added a commit that referenced this issue Oct 8, 2021
@hyejungg
Merge pull request #17 from comento-backend-camp/feature/11
867c3a4 
[feat:#11] 좌석 예약 동시성 코드 추가, BookingHistory 관련 코드 이동, enum 패키지 생성
hyejungg added a commit that referenced this issue Oct 8, 2021
@hyejungg
[fix:#11] 필요 없는 lock 지정문 삭제
f2d4c9f
hyejungg added a commit that referenced this issue Oct 8, 2021
@hyejungg
Merge pull request #19 from comento-backend-camp/feature/11
a1cbe50 
[fix:#11] 필요 없는 lock 지정문 삭제
@hyejungg hyejungg linked a pull request Jan 8, 2022 that will close this issue
[feat#11] 공연 좌석 정보 조회, 공연 좌석 예약 (동시성 해결 X) #15
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hyejungg hyejungg

Labels
feature 기능 구현
Projects
None yet
Milestone
예약 관련 기능 구현
1 participant
@hyejungg