ehBeak created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:21am

😅

View on GitHub

ehBeak created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:21am

View on GitHub

youngsu5582 pushed 1 commit to j-i-h-o woowacourse/java-coupon

October 27, 2024 5:19am

• [1단계 - DB 복제와 캐시] 땡이(정지호) 미션 제출합니다. (#35) * feat: 도메인 구현 * feat: DataSource 라우팅 구현 * feat: 복제 지연이 없는 메서드 지원 ... ec0bebe

View on GitHub

youngsu5582 closed a pull request on woowacourse/java-coupon

October 27, 2024 5:19am

[1단계 - DB 복제와 캐시] 땡이(정지호) 미션 제출합니다.

조이썬 안녕하세요! 처음 인사드리네요 😊 이번 미션에서 복제 지연을 해결하기 위해 고려한 방법은 아래와 같습니다. 1. 별도의 캐시 서버를 두거나 로컬 캐시 사용 1. 즉시 복제 보장의 필요성보다 캐시 서버를 구축하고 관리하는 비용이 더 클 것 2. 로컬 캐시는 WAS가 다중화 된다면 적절히 사용하기 어려울 것임 2. 쿠폰 ...

youngsu5582 created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:18am

View on GitHub

youngsu5582 created a comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:18am

> 부하가 분산되지 않고, 하나의 writerDB가 모든 부하를 받게 됩니다. 이를 해소하기 위한 방법으로는 writerDB에서 바로 데이터를 찾기 전에 readerDB를 먼저 조회해 데이터가 존재하는지 확인하도록 변경할 수 있을 것 같습니다. 사실, 해당 부분이 정말 어려운 거 같아요. 이를 위해서 캐시가 나온거 같습니다. 🙂 ( Wri...

View on GitHub

youngsu5582 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:15am

사실은, 중복된 메소드를 제거하고 싶어서 건의한거라 어떤 방법이든 좋은거 같았어요. ```java @Component public class WriteExecution { // 제네릭 타입 T를 추가 @Transactional public <T> T execute(Supplier<T> supplier) { ...

View on GitHub

youngsu5582 created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:15am

View on GitHub

leegwichan pushed 1 commit to shin-jisong woowacourse/java-coupon

October 27, 2024 5:14am

• [1단계 - 복제 지연] 시소(신지송) 미션 제출합니다. (#52) * feat: read & write DB 설정 * feat: Coupon 구현 * style: 패키지 변경 * feat:... b012598

View on GitHub

leegwichan closed a pull request on woowacourse/java-coupon

October 27, 2024 5:14am

[1단계 - 복제 지연] 시소(신지송) 미션 제출합니다.

안녕하세요 커찬 👍 오랜만이에요! 이제 찐 마지막 미션을 첫 미션 시절을 함께 한 `재귀함수`와 같이 하게 되어서 좋네요 👊 저는 복제 지연을 해결하기 위해 READ로 먼저 조회 후 결과가 없으면 WRITE 조회를 해 보는 전략을 사용했어요 쿠폰이라는 도메인을 고려할 때 복제 지연이 많이 발생할 가능성이 다른 도메인보다는 낮으며 따라서 성능...

leegwichan created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:11am

> 반면, 읽기와 쓰기 서비스를 분리하면 각 서비스가 자체적으로 데이터 소스를 인식하는 것이 전자의 코드보다는 자연스럽다고 느꼈습니다 예를 들어, 쓰기 서비스는 원래 쓰기 데이터베이스와 연결되어 있으므로 트랜잭션 설정 문제 등을 고려할 필요 없이, 작업에만 집중할 수 있습니다 저도 시소의 코드를 보면서, 객체 자체는 많아져 복잡성이 증가하겠지만...

View on GitHub

leegwichan created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:14am

안녕하세요 시소~ 커찬입니다 (:시소미소:) 더 이야기하고 싶은 점은 많지만 우리는 2단계까지 나아가야 하기에, 이만 머지하도록 하겠습니다. 일부 코멘트를 남겼으니 확인해주시고, 다음 PR에 코멘트 달아주시길 부탁하겠습니다. 모르는 점이나 제 코멘트의 이상한 점이 있다면 언제든지 DM 주세요~

View on GitHub

leegwichan created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:14am

안녕하세요 시소~ 커찬입니다 (:시소미소:) 더 이야기하고 싶은 점은 많지만 우리는 2단계까지 나아가야 하기에, 이만 머지하도록 하겠습니다. 일부 코멘트를 남겼으니 확인해주시고, 다음 PR에 코멘트 달아주시길 부탁하겠습니다. 모르는 점이나 제 코멘트의 이상한 점이 있다면 언제든지 DM 주세요~

View on GitHub

zeus6768 created a comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:02am

코멘트에 답변을 달고 보니 질문에 대한 답변을 빼먹었네요!!! 초롱이 해준 질문에 대답해볼게요 ㅎㅎ --- > 제우스가 해당 방법을 선택한 이유는 가장 간단하게 구현할 수 있는 방법이기 때문이라고 하셨는데, 그렇다면 다른 방법으로는 어떤 방법들을 생각해보셨나요?? 다른 방법으로는 1. `Thread.sleep` 메서드를 사용해 쿠...

View on GitHub

youngsu5582 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 5:02am

해당 부분은 저도 아직 명확하게 기준을 잘 못잡는거 같아서 질문했습니다. 장점이나 단점들도 그렇게 명확하지 않다는 생각도 들었고요. ( 어차피, ID 하나 자동으로 생성하는게 그렇게 부하를 줄까? ) ( 클러스터 인덱스일때 문제면 논-클러스터 DB 를 사용하면 무조건 UUID 가 괜찮을까? ) ( 외부에 노출된다고 정말 유의미하게 문제가 되...

View on GitHub

youngsu5582 created a review on a pull request on woowacourse/java-coupon

October 27, 2024 5:02am

View on GitHub

youngsu5582 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:58am

해당 부분은 저도 프로젝트에서 WRITE 로 지정 했습니다.🥲 근데, 생각해보면 대부분의 작업은 READ 가 차지합니다. ( 파레토 법칙 ) 개발자가 실수하는 부분은 대부분 READ 일 가능성이 있지 않을까? 라는 생각도 들어서 질문했습니다. ( 간단한 메소드들 - `LoginArgumentResolver` 에서 검증하는 부분에서 트랜잭션을 ...

View on GitHub

youngsu5582 created a review on a pull request on woowacourse/java-coupon

October 27, 2024 4:58am

View on GitHub

alstn113 pushed 1 commit to mzeong woowacourse/java-coupon

October 27, 2024 4:51am

• [1단계 - DB 복제와 캐시] 제리(김민정) 미션 제출합니다. (#67) * docs: 1단계 요구사항 작성 * feat: `Coupon` 생성 * feat: `MemberCoupon` 생성 ... ced2f2a

View on GitHub

alstn113 closed a pull request on woowacourse/java-coupon

October 27, 2024 4:51am

[1단계 - DB 복제와 캐시] 제리(김민정) 미션 제출합니다.

안녕하세요 구름! 제리입니다 리뷰로 처음 인사드리는 것 같아요 반갑습니다 👋 저는 복제 지연으로 인한 이슈를 해결하기 위해 **쿠폰 조회에 실패하면 Writer DB에서 한 번 더 조회를 수행**했습니다. - 조회 시 지연 시간만큼 기다렸다 응답하는 방법과 다르게 즉각적인 데이터 일관성을 유지하면서도 단순한 구현이 가능합니다. - Wri...

alstn113 created a comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:51am

반동기 복제 방식의 장단점 체크하고 가시면 좋을 것 같아요! 반동기 복제 방식은 비동기 복제 방식과 다르게 레플리카가 변경 내역을 전달 받았음을 확인할 때까지 기다리므로 느려질 수 있을 것 같아요. 참고: https://hudi.blog/mysql-replication-synchronous-type/#반동기-복제-방식

View on GitHub

mzeong created a comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:42am

실제로는 아래와 같은 것들이 복제 지연의 원인이 될 수 있군요! 1. 장기 실행 Query (예. 기본 인스턴스에서 변경을 시작하고 실행하는 데 1시간, 복제본에서 변경 완료하는 데 1시간이 걸린다면 완료 시점의 총 지연은 약 2시간) 2. Write 쿼리량 증가: 트래픽이 증가하거나 특정 배치 작업으로 인한 쓰기 작업이 많아지면서 발생 3. ...

View on GitHub

zeus6768 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:01am

완전히 동의해요~ 명확한 이름으로 변경하는 김에 MemberName을 만들고 Member에 name 필드도 추가했어요. 😁

View on GitHub

zeus6768 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:04am

일관성 있는 코드 작성을 위해 삭제했습니다... ㅎㅎ,,,ㅋㅋ!!

View on GitHub

zeus6768 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:29am

학습을 위해 이렇게 구체적인 질문을 남겨줘서 고마워요! Spring은 @Transactional이 적용된 메서드를 호출할 때, 해당 메서드를 직접 호출하지 않고 프록시 객체를 통해 호출하는데요. 프록시 객체는 메서드 호출 전후로 트랜잭션 경계를 설정해요. 그런데 같은 클래스 내에서 메서드를 호출하면 프록시 객체를 경유하지 않기 때문...

View on GitHub

zeus6768 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:21am

`REQUIRES_NEW` 속성이 부여된 메서드는 호출될 때 기존의 트랜잭션과 별도의 새로운 트랜잭션을 시작해요. 유의할 점은 현재 이 애너테이션의 readOnly 속성이 기본값이라는 건데요. 기본값은 false 이므로 이 메서드를 실행하면 `DataSourceConfig`의 설정에 따라 WRITER DB에서 트랜잭션을 실행할 거에요. ...

View on GitHub

zeus6768 created a review comment on a pull request on woowacourse/java-coupon

October 27, 2024 4:10am

급하게 구현하느라 미처 신경쓰지 못한 것 같아요. 초롱의 말대로 별도 메서드로 분리했어요!

View on GitHub

zeus6768 created a review on a pull request on woowacourse/java-coupon

October 27, 2024 4:33am

View on GitHub

dwax1324 pushed 1 commit to dobby-kim woowacourse/java-coupon

October 27, 2024 4:27am

• [1단계 - DB 복제와 캐시] 도비(김도엽) 미션 제출합니다. (#78) * chore: 초기 DB 설정 변경 * chore: DB read-write 분리 설정 추가 * feat(Coupon):... 986580f

View on GitHub

dwax1324 closed a pull request on woowacourse/java-coupon

October 27, 2024 4:26am

[1단계 - DB 복제와 캐시] 도비(김도엽) 미션 제출합니다.

안녕하세요 종이 :> 또 만나뵙게 됐네요! 제가 저번주 화수목에 동원 훈련을 다녀오느라 미션 제출이 늦었습니다 이번 미션은 다소 설정부터 복잡한 부분이 있어서 조금 시간이 걸렸네요ㅎㅎ 우선, 지연 복제에 대한 문제는 `LazyConnectionDataSourceProxy`로 인해 해결하였습니다! 하지만 성능 저하나 데이터 일관성 문...