Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
unifolio0 pushed 1 commit to masonkimseoul woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 메이슨(김경원) 미션 제출합니다. (#58) * feat: 읽기, 쓰기용 DataSource 분리 * docs: 기능 요구사항 정리 * feat: CouponName... 5dab655
unifolio0 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 메이슨(김경원) 미션 제출합니다.
안녕하세요 비토! 1단계 미션 완료하여 리뷰 요청합니다. 오랜만에 도메인부터 하나씩 작성하니 신선하고 재밌네요. 저는 Redis나 캐시를 쓰는 방법, 반동기로 구현하는 방법을 우선적으로 고려해봤는데요, 캐시를 쓸만큼 쿠폰 생성 후 빠르게 조회를 할 일이 적을 것 같아 선택하지 않았습니다. 발생할 수 있는 트래픽이나 서버 성능에 대한 조건은 ...
unifolio0 created a review comment on a pull request on woowacourse/java-coupon
1. 확인해보니 `CouponName`이나 `DiscountAmount`와 같은 다른 클래스에는 `equals`와 `hashCode`가 없는데 잘 작동하나요?🤔 2. 보통 `Entity`의 경우 `id`를 통해 동등성을 판단한다고 알고 있는데 혹시 위처럼 구현한 이유가 있을까요?
unifolio0 created a review comment on a pull request on woowacourse/java-coupon
**DataSourceRouter** ``` @Override protected Object determineCurrentLookupKey() { if (TransactionSynchronizationManager.isCurrentTransactionReadOnly()) { System.out...
unifolio0 created a review on a pull request on woowacourse/java-coupon
데모데이가 끝나도 여전히 바쁘네요😭 마감기한이 얼마 안 남아서 빠르게 머지합니다! 질문 몇개 남겼으니 확인하고 다음 단계에 반영해주세요! 1단계 수고하셨습니다!
unifolio0 created a review on a pull request on woowacourse/java-coupon
데모데이가 끝나도 여전히 바쁘네요😭 마감기한이 얼마 안 남아서 빠르게 머지합니다! 질문 몇개 남겼으니 확인하고 다음 단계에 반영해주세요! 1단계 수고하셨습니다!
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
le2sky closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 이상(이건상) 미션 제출합니다.
안녕하세요 아톰 만나서 반갑습니다 😄 1단계는 source 와 replica 사이에 복제 지연 1초가 발생하는 상황을 어떻게 해결할 것인가 하는 미션이었어요. 첫 번째로 가장 단순히 처리하는 방법은 source DB에 CUD 하는 작업 후에 Thread.sleep()을 사용해서 replica가 복사하는 시간을 벌어주는 방법이라고 생각합니다. 이...
le2sky created a review comment on a pull request on woowacourse/java-coupon
오전 7시에 1대의 서버가 100개의 작업을 한 순간에 동시에 작업해야해요. 이때 각 작업은 비동기로 처리할 수 있는 작업입니다. 어떻게 corePoolSize, maxPoolSize, queueCapacity의 수를 잘 조정해볼 수 있을까요? 이상의 의견이 궁금합니다! 그리고 spring에서 제공하는 ThreadPoolTaskExecutor가 ...