Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ehBeak created a review comment on a pull request on woowacourse/java-coupon
`type(HikariDataSource.class)`를 명시하지 않으면, `application.properties`나 `application.yml` 설정에서 `spring.datasource.type`에 따라 데이터소스 구현 클래스가 결정됩니다. 그렇지만 명시적으로 설정해 주는 것이 더 안정적이고 일관성을 지킨다고 생각해서 이렇게 작성했습니다!
ehBeak created a review comment on a pull request on woowacourse/java-coupon
`name`은 스프링 빈 이름을 지정하는 값입니다. 만약 `name`을 생략하면 메서드 이름을 빈 이름으로 등록하게 됩니다. 여기서는 `routeDataSource`가 빈 이름으로 등록되겠네요! 그런데 메서드 명이 상수 문자열과 같기 때문에 빈 이름을 딱히 지정하지 않아도 될 것 같아요😅 그래서 해당 클래스에서 빈 이름을 지정하는 모든 부분은 수정했...
youngsu5582 pushed 1 commit to j-i-h-o woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 땡이(정지호) 미션 제출합니다. (#35) * feat: 도메인 구현 * feat: DataSource 라우팅 구현 * feat: 복제 지연이 없는 메서드 지원 ... ec0bebe
youngsu5582 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 땡이(정지호) 미션 제출합니다.
조이썬 안녕하세요! 처음 인사드리네요 😊 이번 미션에서 복제 지연을 해결하기 위해 고려한 방법은 아래와 같습니다. 1. 별도의 캐시 서버를 두거나 로컬 캐시 사용 1. 즉시 복제 보장의 필요성보다 캐시 서버를 구축하고 관리하는 비용이 더 클 것 2. 로컬 캐시는 WAS가 다중화 된다면 적절히 사용하기 어려울 것임 2. 쿠폰 ...
youngsu5582 created a comment on a pull request on woowacourse/java-coupon
> 부하가 분산되지 않고, 하나의 writerDB가 모든 부하를 받게 됩니다. 이를 해소하기 위한 방법으로는 writerDB에서 바로 데이터를 찾기 전에 readerDB를 먼저 조회해 데이터가 존재하는지 확인하도록 변경할 수 있을 것 같습니다. 사실, 해당 부분이 정말 어려운 거 같아요. 이를 위해서 캐시가 나온거 같습니다. 🙂 ( Wri...
youngsu5582 created a review comment on a pull request on woowacourse/java-coupon
사실은, 중복된 메소드를 제거하고 싶어서 건의한거라 어떤 방법이든 좋은거 같았어요. ```java @Component public class WriteExecution { // 제네릭 타입 T를 추가 @Transactional public <T> T execute(Supplier<T> supplier) { ...
leegwichan pushed 1 commit to shin-jisong woowacourse/java-coupon
- [1단계 - 복제 지연] 시소(신지송) 미션 제출합니다. (#52) * feat: read & write DB 설정 * feat: Coupon 구현 * style: 패키지 변경 * feat:... b012598
leegwichan closed a pull request on woowacourse/java-coupon
[1단계 - 복제 지연] 시소(신지송) 미션 제출합니다.
안녕하세요 커찬 👍 오랜만이에요! 이제 찐 마지막 미션을 첫 미션 시절을 함께 한 `재귀함수`와 같이 하게 되어서 좋네요 👊 저는 복제 지연을 해결하기 위해 READ로 먼저 조회 후 결과가 없으면 WRITE 조회를 해 보는 전략을 사용했어요 쿠폰이라는 도메인을 고려할 때 복제 지연이 많이 발생할 가능성이 다른 도메인보다는 낮으며 따라서 성능...
leegwichan created a review comment on a pull request on woowacourse/java-coupon
> 반면, 읽기와 쓰기 서비스를 분리하면 각 서비스가 자체적으로 데이터 소스를 인식하는 것이 전자의 코드보다는 자연스럽다고 느꼈습니다 예를 들어, 쓰기 서비스는 원래 쓰기 데이터베이스와 연결되어 있으므로 트랜잭션 설정 문제 등을 고려할 필요 없이, 작업에만 집중할 수 있습니다 저도 시소의 코드를 보면서, 객체 자체는 많아져 복잡성이 증가하겠지만...
leegwichan created a review on a pull request on woowacourse/java-coupon
안녕하세요 시소~ 커찬입니다 (:시소미소:) 더 이야기하고 싶은 점은 많지만 우리는 2단계까지 나아가야 하기에, 이만 머지하도록 하겠습니다. 일부 코멘트를 남겼으니 확인해주시고, 다음 PR에 코멘트 달아주시길 부탁하겠습니다. 모르는 점이나 제 코멘트의 이상한 점이 있다면 언제든지 DM 주세요~
leegwichan created a review on a pull request on woowacourse/java-coupon
안녕하세요 시소~ 커찬입니다 (:시소미소:) 더 이야기하고 싶은 점은 많지만 우리는 2단계까지 나아가야 하기에, 이만 머지하도록 하겠습니다. 일부 코멘트를 남겼으니 확인해주시고, 다음 PR에 코멘트 달아주시길 부탁하겠습니다. 모르는 점이나 제 코멘트의 이상한 점이 있다면 언제든지 DM 주세요~
zeus6768 created a comment on a pull request on woowacourse/java-coupon
코멘트에 답변을 달고 보니 질문에 대한 답변을 빼먹었네요!!! 초롱이 해준 질문에 대답해볼게요 ㅎㅎ --- > 제우스가 해당 방법을 선택한 이유는 가장 간단하게 구현할 수 있는 방법이기 때문이라고 하셨는데, 그렇다면 다른 방법으로는 어떤 방법들을 생각해보셨나요?? 다른 방법으로는 1. `Thread.sleep` 메서드를 사용해 쿠...
youngsu5582 created a review comment on a pull request on woowacourse/java-coupon
해당 부분은 저도 아직 명확하게 기준을 잘 못잡는거 같아서 질문했습니다. 장점이나 단점들도 그렇게 명확하지 않다는 생각도 들었고요. ( 어차피, ID 하나 자동으로 생성하는게 그렇게 부하를 줄까? ) ( 클러스터 인덱스일때 문제면 논-클러스터 DB 를 사용하면 무조건 UUID 가 괜찮을까? ) ( 외부에 노출된다고 정말 유의미하게 문제가 되...
youngsu5582 created a review comment on a pull request on woowacourse/java-coupon
해당 부분은 저도 프로젝트에서 WRITE 로 지정 했습니다.🥲 근데, 생각해보면 대부분의 작업은 READ 가 차지합니다. ( 파레토 법칙 ) 개발자가 실수하는 부분은 대부분 READ 일 가능성이 있지 않을까? 라는 생각도 들어서 질문했습니다. ( 간단한 메소드들 - `LoginArgumentResolver` 에서 검증하는 부분에서 트랜잭션을 ...
alstn113 pushed 1 commit to mzeong woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 제리(김민정) 미션 제출합니다. (#67) * docs: 1단계 요구사항 작성 * feat: `Coupon` 생성 * feat: `MemberCoupon` 생성 ... ced2f2a
alstn113 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 제리(김민정) 미션 제출합니다.
안녕하세요 구름! 제리입니다 리뷰로 처음 인사드리는 것 같아요 반갑습니다 👋 저는 복제 지연으로 인한 이슈를 해결하기 위해 **쿠폰 조회에 실패하면 Writer DB에서 한 번 더 조회를 수행**했습니다. - 조회 시 지연 시간만큼 기다렸다 응답하는 방법과 다르게 즉각적인 데이터 일관성을 유지하면서도 단순한 구현이 가능합니다. - Wri...
alstn113 created a comment on a pull request on woowacourse/java-coupon
반동기 복제 방식의 장단점 체크하고 가시면 좋을 것 같아요! 반동기 복제 방식은 비동기 복제 방식과 다르게 레플리카가 변경 내역을 전달 받았음을 확인할 때까지 기다리므로 느려질 수 있을 것 같아요. 참고: https://hudi.blog/mysql-replication-synchronous-type/#반동기-복제-방식
mzeong created a comment on a pull request on woowacourse/java-coupon
실제로는 아래와 같은 것들이 복제 지연의 원인이 될 수 있군요! 1. 장기 실행 Query (예. 기본 인스턴스에서 변경을 시작하고 실행하는 데 1시간, 복제본에서 변경 완료하는 데 1시간이 걸린다면 완료 시점의 총 지연은 약 2시간) 2. Write 쿼리량 증가: 트래픽이 증가하거나 특정 배치 작업으로 인한 쓰기 작업이 많아지면서 발생 3. ...
zeus6768 created a review comment on a pull request on woowacourse/java-coupon
완전히 동의해요~ 명확한 이름으로 변경하는 김에 MemberName을 만들고 Member에 name 필드도 추가했어요. 😁
zeus6768 created a review comment on a pull request on woowacourse/java-coupon
일관성 있는 코드 작성을 위해 삭제했습니다... ㅎㅎ,,,ㅋㅋ!!