Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
사실 상수 분리, 에러 객체 분리, 원시값 포장을 적용하고 싶었으나 시간 관계상의 문제가 컸어요! 현재 미션에서 집중할 부분이 DB 복제 지연 문제라 생각하여... 복제 지연에 집중했는데 해당 조건들이 있는 이상 구현을 하는 것은 필요한 것 같아 딱 충족하도록 구현을 하였습니다 2단계 PR를 내고 진행해 볼까 싶어요!
hoyeonyy created a comment on a pull request on woowacourse/java-coupon
> 복제 지연 방식은 read 조회 후 없다면, write를 조회하는 방식을 사용하셨군요👍 해당 방식을 사용한 이유가 궁금합니다~ 현재는 가장 합당한 방법이라 생각했습니다! 캐시 서버와 같은 다른 데이터베이스를 사용하지 않고 현재 사용할 수 있는 최선을 찾은 것 같아요! 물론 특정 상황에서는 트래픽이 전부 write로 몰릴 수 있는 가능성은 있...
nayonsoso created a review comment on a pull request on woowacourse/java-coupon
이 부분은 제가 '모든 읽기 요청'을 강조해서, 헷갈리게 질문을 드렸던 것 같아요😅 (스미마셍..) 제 질문의 의도는, '로컬 캐시 → 리모트 캐시로 바뀌게 되면 추가적인 네트워크 오버헤드가 발생할텐데 그럼에도 현 구조를 유지할 것인가?' 에 대한 내용이었습니다 ㅎㅎ 로컬 캐시에서 얻는 '빠르다'는 이점이, 리모트 캐시에서는 흐려질 수 ...
J-I-H-O created a comment on a pull request on woowacourse/java-coupon
조이썬 안녕하세요! 정말 오랜만에 돌아왔습니다. 요것저것 찾아보며 최대한 답글 남겨두었는데, 모호하거나 이상한 부분이 많을 것 같아요. 많은 가르침 부탁드립니다 😅 > 즉시 복제가 보장되어야 하는 로직이라면 바로 writer DB를 사용하도록 설정 때문에 쿠폰을 발급하고, 쿠폰을 조회하는 연산을 모두 WRITE DB 로 쏘게 되었습니다. 이...
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
좀 더 간결한 코드네요! 두 코드의 명확한 장단점을 못 느껴서 일단 커찬의 것을 반영해 보았어요 추후 다음 단계를 진행하며 체감을 할 수 있게 될지도...
hoyeonyy created a review comment on a pull request on woowacourse/java-coupon
빨리 하다보니 놓친 조건들이 있네요,,ㅠㅠ
donghoony created a comment on a pull request on woowacourse/java-coupon
안녕하세요 마크! 마크의 조언대로 구현해보려고 했지만 생각보다 어려워 몇 가지 이유를 대 보았습니다. 확인해보시고 의견 남겨주세요! 데모데이도 고생 많으셨습니다 ❤️
donghoony created a review comment on a pull request on woowacourse/java-coupon
구현해보려고 시도했는데, 생각보다 난관에 봉착했습니다 😢 기존 분기는 `@Transactional` 분기인데다 전체적인 테이블 이름을 캐시에 저장해두었는데요, 마크의 방식을 적용하기 위해서는 서비스 차원에서 어떤 ID를 사용하고 있는지를 알아야 하더라고요. 그 중에서도 ID를 활용하는, 그리고 활용하지 않는 메서드들을 분리하는 것도 어려워서 반...
nayonsoso created a review comment on a pull request on woowacourse/java-coupon
앗 그런데 수정 과정에서 테스트가 깨진 것 같아요🥲 
JINU-CHANG created a review on a pull request on woowacourse/java-coupon
수고많으셨습니다~ 확인사항이 남아있어 RC 남깁니다!
nayonsoso created a review comment on a pull request on woowacourse/java-coupon
> DataSource라는 같은 타입의 빈을 writerDataSource, readerDataSource, routingDataSource, dataSource라는 이름으로 총 4개나 등록하고 있으니, 스프링컨테이너에서 각 빈을 생성할 때 어떤 빈을 먼저 생성해야 DI 과정에서 문제가 발생하지 않는지 판단하기 어렵나? 스프링에서 빈의 등록 순서...
JINU-CHANG created a review comment on a pull request on woowacourse/java-coupon
테스트 직접 확인했으니 일단 넘어가죠
JINU-CHANG created a review comment on a pull request on woowacourse/java-coupon
reader, writer 문자열을 static 변수로 변수 빼서 공통으로 사용하자는 말이었습니다 :)
JINU-CHANG created a review comment on a pull request on woowacourse/java-coupon
설명이 부족했네요! 예외 메시지를 전달하거나 커스텀 예외를 던지면 예외 사항을 더 빠르게 이해할 수 있을 것 같아요