Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
zeus6768 created a review comment on a pull request on woowacourse/java-coupon
학습을 위해 이렇게 구체적인 질문을 남겨줘서 고마워요! Spring은 @Transactional이 적용된 메서드를 호출할 때, 해당 메서드를 직접 호출하지 않고 프록시 객체를 통해 호출하는데요. 프록시 객체는 메서드 호출 전후로 트랜잭션 경계를 설정해요. 그런데 같은 클래스 내에서 메서드를 호출하면 프록시 객체를 경유하지 않기 때문...
zeus6768 created a review comment on a pull request on woowacourse/java-coupon
`REQUIRES_NEW` 속성이 부여된 메서드는 호출될 때 기존의 트랜잭션과 별도의 새로운 트랜잭션을 시작해요. 유의할 점은 현재 이 애너테이션의 readOnly 속성이 기본값이라는 건데요. 기본값은 false 이므로 이 메서드를 실행하면 `DataSourceConfig`의 설정에 따라 WRITER DB에서 트랜잭션을 실행할 거에요. ...
zeus6768 created a review comment on a pull request on woowacourse/java-coupon
급하게 구현하느라 미처 신경쓰지 못한 것 같아요. 초롱의 말대로 별도 메서드로 분리했어요!
dwax1324 pushed 1 commit to dobby-kim woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 도비(김도엽) 미션 제출합니다. (#78) * chore: 초기 DB 설정 변경 * chore: DB read-write 분리 설정 추가 * feat(Coupon):... 986580f
dwax1324 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 도비(김도엽) 미션 제출합니다.
안녕하세요 종이 :> 또 만나뵙게 됐네요! 제가 저번주 화수목에 동원 훈련을 다녀오느라 미션 제출이 늦었습니다 이번 미션은 다소 설정부터 복잡한 부분이 있어서 조금 시간이 걸렸네요ㅎㅎ 우선, 지연 복제에 대한 문제는 `LazyConnectionDataSourceProxy`로 인해 해결하였습니다! 하지만 성능 저하나 데이터 일관성 문...devhoya97 created a review comment on a pull request on woowacourse/java-coupon
헉 그렇네요.. `%,d`를 사용해서 1000단위로 ,가 붙도록 변경하여 테스트 코드 통과시켰습니다!
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 이 빈은 어떤 목적으로 생성하셨는지 궁금해요! 스캔할 엔티티 패키지를 명시적으로 지정해서 엔티티는 coupon.entity 패키지에 생성한다는 것을 강제할 수도 있고, application.yml에 있는 설정값 및 아래에서 만든 DataSource 빈을 사용한다는 것을 명시하고 싶었어요~!
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 테스트가 꼼꼼해서 좋네요 👍👍👍 감사합니닷 ~🙇
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 현재 방법 외 다른 방법으로 해결하는 방법은 없었을까요? 고민하신 방법들을 공유해주시면 좋을 것 같아요! 캐시는 사용해보지 않아서 고려조차 안했고, 캐시를 사용하지 않는다면 1. 처음부터 쓰기 DB를 사용한다. 2. 실패하는 경우에만 쓰기 DB를 사용한다 로 방법이 나뉠 것 같아요. 현재 규모에서는 1번 방법도 부하가 거의 없을 것이라...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 테스트 깨지는 부분이 있어서 확인 부탁드려요~~~~ (위 테스트 하나만 깨집니다!) 이 부분은 한참 찾아봤는데.. 도커 재시작후 한번 더 해주실 수 있을까요..? 테스트 돌아가는거 확인하고 제출했고 여전히 제 환경에서는 잘 통과해서요..ㅎㅎ;;;
pricelees created a review comment on a pull request on woowacourse/java-coupon
> indent를 줄여보는 것은 어떨까요오 스트림 안에 쓰는 것은 인덴트 1에 해당되지 않나요..? (헷갈려서 여쭤봅니다!) 일단 가독성이 좋지 않다고 판단하신 것 같아서 별도의 변수로 분리는 해두겠습니다~~
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 빈 이름을 메서드명이 아닌 어노테이션을 통해 별도로 설정하신 이유가 궁금합니다! 크게 복잡한게 아니라면 가급적 명시하는 편입니다 ㅎㅎ 말씀주신대로 메서드명만 지정해도 되긴 하지만 name을 지정하는게 더 파악하기 쉽더라구요
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 검증을 Coupon 엔티티 내부가 아닌 service 단에서 진행하신 이유가 궁금해요! 저도 처음에는 엔티티에 검증 로직이 있었는데, 아래 이유들로 별도의 검증을 하도록 수정했어요~ 1. 쿠폰 발급시의 제약 조건은 언제든지 바뀔 수 있다고 생각했습니다. 개인적으로 엔티티 객체는 한번 만들면 가급적 수정하고 싶지 않았습니다. 2. 요구사...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 저의 경우는 타입을 지정하지 않아도 DataSource가 적절하게 생성되는데요! 특정 타입을 지정하신 이유가 있으신가요? application.yml에 type 정보가 있는걸 못봤네요 ㅎㅎ;; 이 부분은 지우겠습니다!
pricelees created a review on a pull request on woowacourse/java-coupon
에버 안녕하세요~ 프로젝트 진행으로 제출이 늦어 죄송합니다 ㅠㅠ 남겨주신 의견들은 하나하나 꼼꼼하게 확인했습니다 ㅎㅎ 답글 달면서 느끼는데 별 생각없이 사용한 코드들이 꽤나 있더라구요! 수정할 부분은 수정했으니 다시 한번 확인 부탁드립니다 ~!
pricelees created a review on a pull request on woowacourse/java-coupon
에버 안녕하세요~ 프로젝트 진행으로 제출이 늦어 죄송합니다 ㅠㅠ 남겨주신 의견들은 하나하나 꼼꼼하게 확인했습니다 ㅎㅎ 답글 달면서 느끼는데 별 생각없이 사용한 코드들이 꽤나 있더라구요! 수정할 부분은 수정했으니 다시 한번 확인 부탁드립니다 ~!
pricelees pushed 1 commit to eun-byeol woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 조조(조은별) 미션 제출합니다. (#62) * chore: 리드미 gitignore 제거 * docs: 1단계 요구사항 정리 * feat: 쿠폰 도메인 생성 *... e3dab2b