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
pricelees closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 조조(조은별) 미션 제출합니다.
상돌🤓 안녕하세요! 조조입니다~ 환경 셋팅부터 정말 쉽지 않았네요..🥲 아직 부족한 부분이 많습니다. 운영환경과 동일해야 한다는 요구사항이 있었지만, `hbm2ddl.auto: validate`으로 두지 못했어요. ddl 파일을 만들어서 시도해보았는데, 생성이 안 되더라구요. 이부분은 더 시도해보겠습니다! 편하게 리뷰주세요! 감사합니다!! ...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> (반영 완료) init.sql 추가하여 docker compose 실행 시 ddl 실행되도록 수정했어요! > 고생하셨습니다 ㅎㅎ > 이 방법도 고려했었는데, 리뷰어에게 불친절한 방법이라 생각해서 하지 않았어요! > 저는 괜찮으니 조조 편한 방법대로 하셔도 됩니다 ㅎㅎ > 그래서 더욱 직접 DDL 작성해야 한다고 생각했어...
pricelees created a review comment on a pull request on woowacourse/java-coupon
> 이 부분에 동의해요. 다만, 필요성에는 공감하지 못했어요. private 메서드로 분리하지 않으면 가독성이 떨어진다고 생각해요. 이 외 다른 어떤 추가적인 문제가 발생할까요?🤔 가독성 면에서는 저도 조조가 말씀하신대로 메서드 분리를 하는게 좋다고 생각합니다 ㅎㅎ 그런데 findByIdWithWriteDB()를 보면 writeDBConnect...
pricelees created a review on a pull request on woowacourse/java-coupon
조조 안녕하세요~ 데모데이가 끝나도 여전히 바쁘시군요 제가 리뷰를 어쩌다보니 꽤나 많이 남기게 되었는데.. 하나하나 꼼꼼히 반영해주셔서 감사합니다! 나머지 이야기는 2단계에서 해도 될 것 같아요~ 고생하셨습니다!
pricelees created a review on a pull request on woowacourse/java-coupon
조조 안녕하세요~ 데모데이가 끝나도 여전히 바쁘시군요 제가 리뷰를 어쩌다보니 꽤나 많이 남기게 되었는데.. 하나하나 꼼꼼히 반영해주셔서 감사합니다! 나머지 이야기는 2단계에서 해도 될 것 같아요~ 고생하셨습니다!
linirini created a comment on a pull request on woowacourse/java-coupon
프린~ 런칭데이까지 바쁘게 보내고, 정신 차리고 보니 벌써 시간이 많이 흘렀네요...ㅎㅎ 리뷰 요청이 많이 늦어졌어요, 죄송합니다..!🥲 리뷰 잘 부탁드립니다-!
linirini created a review comment on a pull request on woowacourse/java-coupon
`@Primary`로 Datasource 의존성을 주입받을 때 LazyConnectionDataSourceProxy가 반환되도록 설정하여 Connection을 받아오는 시점을 지연시킵니다.  ...
linirini created a review comment on a pull request on woowacourse/java-coupon
JPA로 연결되는 DataSource가 하나일 때는 `JpaTransactionManager`가 자동으로 할당 됩니다. 하지만, 지금처럼 두 개 이상의 Datasource를 사용할 때에는 Spring에서 어떤 Datasource와 EntityManagerFactory를 사용해야할지 판단할 수 없어요. 따라서, 위와 같이 직접 TransactionMan...
linirini created a review comment on a pull request on woowacourse/java-coupon
우선은 메서드명으로 구분할 것 같아요! (`getCouponByAdmin`으로 변경했습니다) 추후에 권한 개념(Role)이 도입된다면, 메서드 내부적으로 Role이 ADMIN이 맞는지 검증하는 작업을 추가해볼 것 같아요!