[2단계 - DB 복제와 캐시] 프린(남기범) 미션 제출합니다. #93
Merged
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
slimsha2dy
requested changes
Oct 29, 2024
Comment on lines
+36
to
+37
| @Column(nullable = false) | ||
| private Long memberId; |
There was a problem hiding this comment.
쿠폰은 저장소 분리로 인해 연관관계를 끊은 것으로 이해했는데 멤버도 저장소 분리를 고려해서 연관관계를 끊으신 걸까요?
There was a problem hiding this comment.
넵 멤버도 db를 분리하지 않을까 싶어서 끊었어요!
| @Transactional | ||
| public void issue(Long memberId, Long couponId) { | ||
| Member member = getMember(memberId); | ||
| Coupon coupon = couponService.getCoupon(couponId); |
There was a problem hiding this comment.
서비스 계층이 서비스를 의존하는 건 유지보수성, 확장성, 예측하기 힘든 구조 등의 여러 이유로 지양해야 한다고 생각하긴 하는데요. 그럼에도 이번 과제에서 간단 명료한 구조를 유지하면서 해결하기엔 쉽지 않을 것 같아 고민이 많았습니다.. 그리고 이런 이유로 기능 계층과 구현 게층을 분리하는 이유를 요즘 알 것 같기도 해요 😢
프린은 어떤 이유로 같은 계층 간 의존관계를 허용하게 되었나요?
There was a problem hiding this comment.
캐시를 적용해서 서비스 계층끼리 의존하게 되었어요
지금은 구현 크기가 작아서 허용하고 넘어갔지만 어느 정도 크기가 커지면 미리 분리해볼 것 같아요 땅콩에서는 서비스 계층 위에 파사드 서비스 계층을 하나 더 두어서 해결했었습니다!
Comment on lines
+12
to
+24
| @ExtendWith(DatabaseCleanerExtension.class) | ||
| @SpringBootTest(webEnvironment = WebEnvironment.NONE) | ||
| public abstract class BaseServiceTest { | ||
|
|
||
| @Autowired | ||
| protected CouponRepository couponRepository; | ||
|
|
||
| @Autowired | ||
| protected MemberRepository memberRepository; | ||
|
|
||
| @Autowired | ||
| protected MemberCouponRepository memberCouponRepository; | ||
| } |
| void setUp() { | ||
| member = memberRepository.save(new Member("alpaca")); | ||
| coupon = couponRepository.save( | ||
| new Coupon("알파카 털뭉치 할인", 1_000, 5_000, LocalDate.now(), LocalDate.now(), Category.FOOD)); |
| import org.springframework.boot.test.context.SpringBootTest; | ||
| import org.springframework.boot.test.context.SpringBootTest.WebEnvironment; | ||
|
|
||
| @ExtendWith(DatabaseCleanerExtension.class) |
There was a problem hiding this comment.
이거도 처음 보는데 굉장히 스킬풀하네요
하나 배워갑니다 ㅎㅎ
slimsha2dy
approved these changes
Nov 1, 2024
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.
안녕하세요 알파카 🦙🦙
1단계에서 WAS 이중화를 가정했기 때문에 Redis를 캐시 서버로 사용하고, 캐시에 데이터가 없으면 DB를 조회하는
Look Aside전략을 선택했습니다쿠폰을 생성할 때는 캐시에 저장하지 않기 때문에 캐시 쓰기 전략은 고려하지 않았어요
캐시된 데이터의 TTL은 30분입니다
쿠폰 중에서도 자주 조회되는 쿠폰(ex. BBQ와 같은 프랜차이즈 가게)과 자주 조회되지 않는 쿠폰(ex. 개인 자영업자 가게)이 있을 것이라고 생각했고, 자주 조회되지 않는 쿠폰이 불필요하게 메모리를 점유하지 않도록 하기 위함이에요
자주 조회되는 쿠폰은 TTL이 지속적으로 갱신되기 때문에 TTL을 30분으로 잡더라도 cache miss가 자주 발생하지 않을 것 같고, 더 적절한 TTL 값은 cache miss를 모니터링하면서 찾아보지 않을까 싶어요
3단계는 진행하지 않을 예정이라서 2단계가 마지막이 될 것 같습니다 이번 리뷰도 감사합니다 👍🏻