Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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이 맞는지 검증하는 작업을 추가해볼 것 같아요!
linirini created a review comment on a pull request on woowacourse/java-coupon
현재는 application.yml에서 ddl 전략으로 `create`를 사용하고 있어요! 또한, docker-compose에서 컨테이너가 시작될 때 초기화될 데이터베이스의 이름을 지정해놓았습니다. 그렇기 때문에 docker compose up을 할 때 coupon이라는 데이터베이스가 생성되고, 애플리케이션을 실행시킬 때 기존의 테이블을 제거 후...
jimi567 opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 위브(김기범) 미션 제출합니다.
안녕하세요. 로빈 2단계 구현이 된 것 같아서 리뷰 요청드립니다.!! 변경 사항은 다음과 같습니다. 1. MemberCoupon 변경 >[미션 설명](https://techcourse.woowahan.com/s/Cjiz6fNL/lt/R6TAnPXF)에 있는 것처럼 쿠폰과 회원에게 발급된 쿠폰의 저장소가 분리될 예정이므로, 두 테이블을 조인...