Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
HoeSeong123 closed a pull request on woowacourse/java-coupon
[1단계 - 복제 지연] 제우스(신재우) 미션 제출합니다.
안녕하세요 초롱~! ⚡⚡⚡ 먼저 다음의 서비스 코드에서 복제 지연을 확인했어요.  
ehBeak created a review comment on a pull request on woowacourse/java-coupon
오 그렇네요! 꼼꼼한 리뷰 감사합니다 🥹🥹 넵 `TransactionSynchronizationManager.isActualTransactionActive()`을 확인하여, 중첩트랜잭션일 경우 `source`에 바로 접근하고 그렇지 않을 경우 `replica`에 접근하도록 수정했습니다 :)
HoeSeong123 created a review on a pull request on woowacourse/java-coupon
우형, DH 준비로 바쁠텐데도 정말 꼼꼼하게 잘해주셨네요!!! 이번 단계는 여기서 마무리하도록 하겠습니다ㅎㅎ 바쁘겠지만 2단계도 화이팅입니다!! 이제 진짜 우테코 마지막 미션!!!!
HoeSeong123 created a review on a pull request on woowacourse/java-coupon
우형, DH 준비로 바쁠텐데도 정말 꼼꼼하게 잘해주셨네요!!! 이번 단계는 여기서 마무리하도록 하겠습니다ㅎㅎ 바쁘겠지만 2단계도 화이팅입니다!! 이제 진짜 우테코 마지막 미션!!!!
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
> 레모네가 해주신 이야기는 아직 제게 설득이 되지 않았는데요, coupon조회는 사용자(가게 사장님, 일반유저)를 위한 기능입니다. > 쿠폰을 발급하는 건 가게 사장님이고 사장님만을 위한 조회라면 사용자가 비교적 적으니 납득이 되지만, 일반유저도 쿠폰조회 (getCoupon())을 많이 사용합니다. > 이랬을 때, Writer DB에 너무 많은 ...
ehBeak created a review comment on a pull request on woowacourse/java-coupon
`LazyConnectionDataSourceProxy`는 실제 데이터베이스 연결 초기화를 지연하는데 사용합니다. 즉, 실제 데이터베이스 연결이 필요할 때 연결을 시킵니다. 이를 사용하는 이유는 커넥션 자원을 절약하고 트랜잭션에서 읽기/쓰기 분리를 필요로 하는 상황에서 사용하기 위함입니다.
nayonsoso pushed 1 commit to devhoya97 woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 카고(정민호) 미션 제출합니다. (#54) * feat: multi datasource 구성 * docs: 기능 요구사항 정리 * feat: 범위와 단위 조건을 고려... 3088b3a
nayonsoso closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 카고(정민호) 미션 제출합니다.
안녕하세요 산초~ 미션으로 만나니 새롭네요 😄 프로덕션 환경에서 복제 지연으로 인한 문제가 예방하는 방법에 대해 고민해봤는데요. 제가 선택한 방법은 캐시입니다. writer DB에 데이터가 들어가는 동시에 캐시에도 데이터가 들어가므로, reader DB에 복제 지연이 발생하더라도 문제가 없을 것이라 판단했어요. 스프링부트에서 제공하는 기본적...
nayonsoso created a review on a pull request on woowacourse/java-coupon
테스트 통과 확인했습니다😊 이번 단계에서 이야기할 부분은 다 한 것 같아서, 이만 approve & merge 합니다. 다음 단계도 파이팅입니다!!
ehBeak created a review comment on a pull request on woowacourse/java-coupon
네 등록한 값과 키를 맞추었습니다~! 말씀 주신 것 처럼 추가로 로그도 추가했습니다 🫡
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
먼저 도메인과 엔티티를 분리한 이유를 말씀드리겠습니다! 도메인과 엔티티가 하나의 클래스에서 관리될 경우, 객체지향적인 코드를 작성하기 어렵다고 생각했습니다. 쿠폰의 `발급 시작일`과 `발급 종료일`을 `발급 기간(ValidityPeriod)`이라는 하나의 객체로 묶고, 발급 시작일과 발급 종료일의 자료형은 `Date` 객체인 상황을 예시로 들어...
Kimprodp pushed 1 commit to hoyeonyy woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 호기(조호연) 미션 제출합니다. (#83) * feat: 맴버, 맴버 쿠폰 생성 * feat: 쿠폰 관련 도메인 생성 * feat: CouponRepository 생... 8544107
Kimprodp closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 호기(조호연) 미션 제출합니다.
안녕하세요 테드! 캠퍼스 바뀐 이후에 오랜만에 뵙는 것 같아요! 최근에 몸이 안좋아서 늦게 드려 죄송합니다,,🥲 최종 데모데이가 얼마 남지 않아 바쁘실 것 같은데 시간나실때 천천히 해주세요!
Kimprodp created a comment on a pull request on woowacourse/java-coupon
도메인 검증과 테스트 추가해주셨군요! 다음 단계 진행해도 될 것 같아요. 테스트 중 하나 수정이 필요한 부분이 있는데 단순한 사항이라 일단은 다음 단계에서 반영 부탁해요! CouponServiceTest 에서 replicationLagTest() 가 쿠폰에 검증을 추가하면서 해당 검증에 걸리고 있어요 금액만 수정해주시면 될 것 같아요.
ehBeak created a review comment on a pull request on woowacourse/java-coupon
없습니다..😅 일관성을 지키기 위해 모든 도메인에서 생성자에 id를 초기화하지 않는 방향으로 수정했습니다!
ehBeak created a review comment on a pull request on woowacourse/java-coupon
네 맞습니다. 사실 지금으로는 한글 이름이 의미 없긴 하지만, 요구사항에 한글 이름으로 카테고리를 적어주셔서 이렇게 추가했습니다. 물론 상수명을 한글로 적어도 되겠지만, 보통은 영어로 적기 때문에 한글 이름을 필드로 두고 영어로 상수명을 적었습니다~
ehBeak created a review comment on a pull request on woowacourse/java-coupon
name.isBlank()을 실행할 때, name이 null 값이라서 예외가 발생하네요. 예외 처리를 하고 테스트 또한 추가했습니다~~