Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
eunjungL pushed 1 commit to youngsu5582 woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 조이썬(이영수) 미션 제출합니다. (#69) * test: UUID, LocalDateTime 기본적인 테스트 * feat: 외부 파일에서 설정 불러오는 기능 구현 ... 433451a
eunjungL closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 조이썬(이영수) 미션 제출합니다.
안녕하세요 클로버! 이번에도 같은 캠퍼스에서 리뷰어를 만나서 너무 좋네요 🙂 크게, 접근한 부분은 2가지 입니다. ## 외부 설정을 통한 `Validation Config` 주입 Service 나 Validation 내부에서 값을 초기화 하고 가지는게 아니라 외부에서 설정을 통해 주입 하게 했습니다. 이렇게 한 이유는 `검증 ...eunjungL created a review on a pull request on woowacourse/java-coupon
안녕하세요, 조이썬! 클로버입니다. 리뷰 중에 질문이 대다수였는데 꼼꼼하게 답변 달아주셔서 감사해요. 코드에 리뷰 사항이 반영된 것도 확인했어요! 다만 [놓친 코멘트](https://github.com/woowacourse/java-coupon/pull/69#discussion_r1807655967)가 하나 있는 것 같아 리마인드 드려요....
eunjungL created a review comment on a pull request on woowacourse/java-coupon
> 단점으로는 잘못 쓰게 되면, 결국 기존 더러운 코드들을 그냥 하위 계층으로 밀어 넣는거 밖에 안되는거 같아요 🥲 이 부분은 파사드 서비스를 도입한 저희 팀에서도 느끼고 있는 부분이라 많이 공감되네요 😢 파사드는 기존의 복잡한 코드를 상위 서비스에 모두 박아둬 해결했다는 찜찜함이 남게 되는데 하위 계층의 분리에서도 비슷한 주의점이 있네요....
masonkimseoul created a review comment on a pull request on woowacourse/java-coupon
create이랑 getCoupon에 달아야 할 어노테이션이 서로 바꼈네요. 원래 의도한 바와 다르게 어노테이션을 붙여버렸습니다 😅 그리고 `DataSourceRouter` 클래스에서도 불필요한 분기가 있어서 제거해주었습니다.
masonkimseoul created a review comment on a pull request on woowacourse/java-coupon
트랜잭션이 꼭 필요한 로직과 그렇지 않은 로직을 분리하기 위해 이 방식을 선택했습니다.
dwax1324 created a comment on a pull request on woowacourse/java-coupon
감좌 안녕하세요 도메인 및 테스트 구현해서 리뷰요청 드립니다
tsulocalize opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 우주(김성현) 미션 제출합니다.
안녕하세요, 제제! 저는 이번에 처음으로 레디스를 다뤄본 것 같아요 나중에 시간이 되면 DB에 비해 얼마나 빠른지 테스트도 해보고 싶네요 이번에도 잘 부탁드리겠습니다!tackyu created a comment on a pull request on woowacourse/java-coupon
안녕하세요 위브, 제가 복제 지연에 대해 고민했던 사항에 대해 코멘트 남겨요! sleep 개선 하고 싶었던 이유 1. 정확히 얼마나 sleep 해야하는 지 기준이 모호하다. 2. 지연 발생 여부에 상관 없이 sleep하는 것이 비효율적이라 생각했습니다. 3. 기준도 모호하고, 불필요한 sleep도 발생할 수 있으므로 애플리케이션의 성능 측면...
kunsanglee pushed 1 commit to chooseoyeon woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 도라(추서연) 미션 제출합니다. (#57) * chore: 데이터소스 초기설정 * chore: 쓰기, 읽기 데이터 소스 분리 설정 * docs: 기능 요구사항 추가 ... c9518ef
kunsanglee closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 도라(추서연) 미션 제출합니다.
안녕하세요 이상! 잘부탁드립니다! # 프로젝트 테스트 실행 방식 해당 프로젝트의 테스트 목적은 데이터 변경에 대한 복제 지연 문제 확인이고, 의도한 대로 실행하기 위해선 아래 방식으로 사용하심 됩니다! 1. hdm2ddl.auto 값을 create으로 두고 CouponApplication을 실행해 테이블을 생성한다. 2. hdm2ddl....kunsanglee created a review on a pull request on woowacourse/java-coupon
안녕하세요 도라😇 제가 남긴 코멘트에 답변하신 것을 보니 정말 깊이있게 학습하신것을 알 수 있었어요. 1단계는 이만하면 충분한 것 같아서 2단계 캐시로 넘어가서 진행 해주시면 되겠습니다. 정말 고생 많으셨고 2단계 미션도 기대하겠습니다 화이팅! 💪
kunsanglee created a review comment on a pull request on woowacourse/java-coupon
우와 엄청 깊이있게 학습하셨군요!! 👍👍
Dobby-Kim opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 도비(김도엽) 미션 제출합니다.
안녕하세요 종이 :> 많이 바쁘시죠! 미션 내용을 학습해보면서 적용해보니 조금은 재미가 있었던 것 같습니다. 캐싱 관련하여 구현 내용에 대해 설명드리자면 다음과 같습니다. ### 쿠폰 조회 캐싱 (getCoupon(Long couponId)): `@Cacheable(value = "coupons", key = "#coupo...le2sky pushed 1 commit to kunsanglee woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 이상(이건상) 미션 제출합니다. (#76) * feat: 1단계 - 복제 지연 구현 * refactor: ReadOnlyDataSourceRouter 클래스 추가하여 D... 915f2aa