-
Notifications
You must be signed in to change notification settings - Fork 1
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
[feat] 카트를 조회할 수 있는 Dao를 Redis를 이용해서 구현 #135
Merged
Dr-KoKo
merged 6 commits into
feature/127_Dr-KoKo_구매자는_카트를_조회할_수_있다
from
feature/127_Hyeon-Uk_구매자는_카트를_조회할_수_있다
Aug 22, 2024
The head ref may contain hidden characters: "feature/127_Hyeon-Uk_\uAD6C\uB9E4\uC790\uB294_\uCE74\uD2B8\uB97C_\uC870\uD68C\uD560_\uC218_\uC788\uB2E4"
Merged
[feat] 카트를 조회할 수 있는 Dao를 Redis를 이용해서 구현 #135
Dr-KoKo
merged 6 commits into
feature/127_Dr-KoKo_구매자는_카트를_조회할_수_있다
from
feature/127_Hyeon-Uk_구매자는_카트를_조회할_수_있다
Aug 22, 2024
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
Dr-KoKo
changed the base branch from
main
to
feature/127_Dr-KoKo_구매자는_카트를_조회할_수_있다
August 22, 2024 14:18
Dr-KoKo
approved these changes
Aug 22, 2024
Dr-KoKo
merged commit Aug 22, 2024
ff160c3
into
feature/127_Dr-KoKo_구매자는_카트를_조회할_수_있다
1 check passed
Dr-KoKo
added a commit
that referenced
this pull request
Aug 23, 2024
* [feat] 조회하고자 하는 카트가 없을 때 발생시킬 예외 구현 * [feat] 구매자의 카트를 조회하는 Dao * [feat] 카트 조회 앤드포인트 * [feat] 카트를 조회할 수 있는 Dao를 Redis를 이용해서 구현 (#135) * [test] menu 더미를 저장하고 반환하는 메서드 추가 및 상점을 24시간 열려있도록 수정 * [feat] getter 및 생성자 추가 * [feat] redis용 dao를 임시로 inMemoryCartRepository를 이용해서 구현 * [test] RedisCartDao 테스트코드 추가 * [feat] CartItemInfo에도 getter 추가 * [fix] 테스트 이후 데이터를 롤백시키기 위해 @transactional 추가 * [fix] CartResponse의 amount를 Integer로 수정 * [feat] Jpa를 이용한 카트 조회 dao --------- Co-authored-by: 김현욱 <[email protected]>
Dr-KoKo
added a commit
that referenced
this pull request
Aug 29, 2024
* [feat] redis를 이용하여 cart를 저장/조회/삭제하도록 구현 (#134) * [feat] Redis 환경설정 파일 생성 - port 와 host를 받아 connection을 생성 - redisTemplate에 직렬화 도구로 key : string, value = GenericJackson2JsonRedisSerializer 로 설정 * [fix] Cart의 생성자를 protected에서 public으로 변경 * [fix] CaartService에서 customerId의 cart를 가져오는 로직을 supplier로 변경 * [feat] Redis전용 CartEntity와 CartItemEntity를 생성 * [feat] Redis를 이용해서 Cart를 영속하는 레포지토리 구현 * [test] Redis를 이용해서 Cart를 영속하는 레포지토리 테스트 * [fix] ContainerSettingTest 삭제 여러개의 test-container를 구동시키면 포트 충돌로 일단 삭제 * [fix] testcode 가 돌아갈 수 있도록 환경변수 추가, inmemory cart repository를 primary로 수정 * [feat] Cart를 DB에 저장한다 (#133) * [fix] 카트 도메인 엔티티 수정 * [feat] 카트 영속성 엔티티 구현 * [feat] DB를 사용하는 CartRepository 구현체 * [fix] CartService 오류 수정 Transactional 적용 orElse -> orElseGet : 데이터가 존재함에도 불구하고 저장되는 오류 수정 * [fix] 도메인 엔티티로 변환시 id값도 추가 * [refactor] 변수명 변경 * [fix] 생성자 접근제어자 protected * [test] JpaCartRepository 테스트 코드 작성 * [feat] 구매자는 카트를 조회할 수 있다 (#137) * [feat] 조회하고자 하는 카트가 없을 때 발생시킬 예외 구현 * [feat] 구매자의 카트를 조회하는 Dao * [feat] 카트 조회 앤드포인트 * [feat] 카트를 조회할 수 있는 Dao를 Redis를 이용해서 구현 (#135) * [test] menu 더미를 저장하고 반환하는 메서드 추가 및 상점을 24시간 열려있도록 수정 * [feat] getter 및 생성자 추가 * [feat] redis용 dao를 임시로 inMemoryCartRepository를 이용해서 구현 * [test] RedisCartDao 테스트코드 추가 * [feat] CartItemInfo에도 getter 추가 * [fix] 테스트 이후 데이터를 롤백시키기 위해 @transactional 추가 * [fix] CartResponse의 amount를 Integer로 수정 * [feat] Jpa를 이용한 카트 조회 dao --------- Co-authored-by: 김현욱 <[email protected]> * [feat] CartEntity의 CustomerId 변경 String -> UUID * [fix] CartDao 이중 등록 제거 * [fix] CartItemEntity에 id가 null로 갱신되는 문제 해결 Co-authored-by: Dr-KoKo <[email protected]> * [fix] 의존 주입할 Bean을 application.yml에서 관리할 수 있도록 수정 Co-authored-by: Dr-KoKo <[email protected]> * [feat] 중복 주문에 대한 Exception 생성 및 error code 갱신 Co-authored-by: Dr-KoKo <[email protected]> * [test] 중복 주문 동시성 테스트에 사용할 더미데이터 만드는 메서드 추가 Co-authored-by: Dr-KoKo <[email protected]> * [test] 중복 주문 방지에 대한 동시성 테스트 시나리오 추가 Co-authored-by: Dr-KoKo <[email protected]> * [refactor] 검증 부분 주석 추가 * [fix] OneToMany 관계에서 cascade를 사용하기 위해 mappedBy 사용 @joincolumn만 이용하면 CartEntity를 통해CartItemEntity를 영속화시키지 못한다. * [feat] 마이크로미터 앤드포인트 노출 * [feat] 마이크로미터 앤드포인트 노출 * [test] 마이크로미터 앤드포인트 노출 테스트용 yml 수정 * [fix] APIResponseAdvice에 @EnableConfigurationProperties 추가 * [fix] JpaCartDao 오류 수정 fetchOne()의 결과로 리스트가 반환되어 오류가 발생 * [feat] 주문 시점에 배타락을 적용해서 문제를 해결 (#148) * [feat] 카트 조회시 X-lock 적용 * [test] TransactionTemplate를 통해서 Cart를 조회 * [feat] 주문 시점에 분산락을 적용해서 문제를 해결한 버전 (#147) * [feat] cartRepository의 findByCustomerId에 분산락을 적용한 버전 임시 * [feat] 분산락을 잡지 못했을 경우에 던지는 exception을 커스텀해서 던질 수 있도록 수정. * [fix] wait time 을 0초로 변경 - 중복 결제 요청들이 락을 다시 획득하기 위해서 기다릴 필요가 없음 --------- Co-authored-by: donghar <[email protected]> * Revert "[feat] 주문 시점에 배타락을 적용해서 문제를 해결 (#148)" This reverts commit a2b0da5. * [fix] CartRepository를 트랜잭션 내에서 테스트 * [fix] cart 저장소를 redis로 변경 * [fix] Redisson에 의존하는 configuration 이름을 redis로 변경 * [fix] 생성자 통일 두 개로 분리되어 있던 생성자를 하나로 통일 * [fix] redis 설정 통합 * [fix] 컴파일 오류 해결 디폴트 예외 설정의 누락으로 인한 컴파일 오류 해결 * [fix] 테스트시 in_memory 버전의 CartRepository를 사용하게 설정 * [refactor] build.gradle 정리 --------- Co-authored-by: 김현욱 <[email protected]> Co-authored-by: Hyeon-Uk <[email protected]>
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.
💡 다음 이슈를 해결했어요.
Issue Link - #127
✅ 셀프 체크리스트