Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
GIVEN53 opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 프린(남기범) 미션 제출합니다.
안녕하세요 알파카 🦙🦙 1단계에서 WAS 이중화를 가정했기 때문에 Redis를 캐시 서버로 사용하고, 캐시에 데이터가 없으면 DB를 조회하는 `Look Aside` 전략을 선택했습니다 1. 캐시 서버에 데이터가 있는지 확인 (cache hit) 2. 캐시 서버에 없으면 DB에서 데이터 조회 (cache miss) 3. DB에서 조회한 데이터를 ...
J-I-H-O created a review comment on a pull request on woowacourse/java-coupon
> 추가로, 로직이 전부 똑같은데 메소드를 새로 만드는건 비효율적인 것 처럼 느껴집니다. 토미의 접근이나, java-jdbc 에서 좀 더 편리하게 한 거 처럼 이를 해결 할 방법은 없을까요? 요 부분은 어떻게 하면 좋을지 방법이 떠오르지 않네요 🥺 일단 중복되는 로직을 repository로 내려보았습니다. 더 좋은 방법이 있다면 공유 부탁드립...
nayonsoso created a review comment on a pull request on woowacourse/java-coupon
아하 그렇네요😅 지금처럼 하는게 맞는 것 같습니다~
linirini created a review comment on a pull request on woowacourse/java-coupon
지금은 생성자 체이닝에서만 사용되고 있어서 우선 접근 제어자를 변경해보았습니다!
linirini created a review comment on a pull request on woowacourse/java-coupon
DiscountAmount에게 MinimumOrderAmount에 대한 할인율을 계산하는 것을 요청하는 방식으로 리팩터링했습니다. 그렇지만 할인율을 검증하는 것 자체의 책임은 DiscountAmount에게 있지 않다고 생각했어요! 쿠폰마다 할인율 제약이 달라질 수 있다고 생각해서 쿠폰의 책임이라고 생각했습니다! 프린은 어떻게 생각하시나요? + L...
J-I-H-O created a review comment on a pull request on woowacourse/java-coupon
> 여기서 있는 Lag 이 어떤 의미일까요? 🫡🫡 replication lag을 의도했습니다! writerDB에 바로 접근하면 readerDB에 복제되는 지연 시간이 없을 것이라고 생각해 해당 네이밍을 사용했습니다.
J-I-H-O created a review comment on a pull request on woowacourse/java-coupon
조금 더 검색해보니, 분산 환경 관점에서 두 방식을 비교하는 글을 찾아볼 수 있었습니다. auto increment 방식은 중복 id 생성을 막기 위해 별도의 방법을 사용해야 합니다. 예를 들어 id 증가 폭을 조정해 서로 다른 두 DB 서버에서 중복 id가 생성되지 않도록 유일성을 보장하거나, id를 생성해주는 중앙 집중 시스템을 구축해야 합...
J-I-H-O created a review comment on a pull request on woowacourse/java-coupon
auto increment가 익숙한 방식이라는 점이 가장 큰 이유였습니다. 이렇게 생성된 id는 정렬 기준으로 사용하기도 용이하고, UUID에 비해 데이터 크기가 작아 유리하다는 점만 알고 사용해왔어요. 프로젝트를 진행하며 경험적으로 느낀 auto increment 방식의 단점은 외부에서 예측하기 쉽다는 점이었습니다. 데이터를 단건 조회하는...
eunjungL opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 클로버(이은정) 미션 제출합니다.
안녕하세요, 콜리 🥦 ! 클로버입니다. 벌써 두 번째 리뷰 제출이네요~ 마지막 데모데이 준비하느라 바쁘셨을텐데 세심한 리뷰 감사했습니다. ### 캐시 도입 이번에는 요구사항을 맞추기 위해 캐시를 도입했어요. 캐싱 도입을 위해 제가 고려했던 점은 아래와 같아요. - 로컬 캐시와 글로벌 캐시 중 어느 것을 도입할지 고민했는데요, 이후 DB...
Arachneee pushed 1 commit to nak-honest woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 낙낙(이낙헌) 미션 제출합니다. (#53) * chore: DB 관련 세팅 * chore: reader/writer DB 라우팅 설정 * feat: 카테고리, 쿠폰,... 725feda
Arachneee closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 낙낙(이낙헌) 미션 제출합니다.
안녕하세요 백호!! 리뷰어로 만나 뵙게 되어 반갑습니다!! ### 복제 지연을 처리한 방법 저는 쿠폰을 조회할 때 writer에서 읽도록 하여 복제 지연을 해결하도록 하였습니다! 시간이 없어 많은 고민을 하지는 못했지만 다음과 같은 이유 때문이었습니다. 비즈니스적으로 생각했을 때 쿠폰을 생성하자마자 바로 조회하는 것은 관리자일테고,...
Arachneee created a review comment on a pull request on woowacourse/java-coupon
쿠폰 이름만 null 검증이 없는 것이 조금 어색해요.
Arachneee created a review on a pull request on woowacourse/java-coupon
안녕하세요 낙낙! 리뷰를 잘 반영해주셨네요! 머지하고 다음단계를 진행해도 좋을 것 같아요!
Arachneee created a review on a pull request on woowacourse/java-coupon
안녕하세요 낙낙! 리뷰를 잘 반영해주셨네요! 머지하고 다음단계를 진행해도 좋을 것 같아요!
PgmJun opened a pull request on woowacourse/java-coupon
Step2 이든 미션 제출
안녕하세요 폴라~ 이든입니다! 사실 이번 미션은 선택과 집중을 위해 프로젝트에 좀 더 공수를 많이들이느라 제대로 코드를 작성하지 못해서 죄송하단 말씀부터 드리고 싶네요 리뷰를 통해 같이 배워나가는 것이 있어야할텐데 그걸 드리지 못해서 굉장히 죄송하게 생각합니다 ㅠ 그래도 제가 가진 시간내에서 최선을 다했으니 리뷰 잘부탁드립니다! 그리고 현재...
J-I-H-O created a review comment on a pull request on woowacourse/java-coupon
`setDefaultTargetDataSource()`는 LookUpKey에 해당하는 데이터 소스가 없는 경우 사용할 default 데이터 소스를 지정해주는 메서드로 이해했어요. 기본값으로 readerDB를 지정해줄 수도 있었겠지만, 그렇게 되면 읽기-쓰기 작업을 분리하는 replication을 의도대로 사용할 수 없을 것 같습니다. 애플...