Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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)에 있는 것처럼 쿠폰과 회원에게 발급된 쿠폰의 저장소가 분리될 예정이므로, 두 테이블을 조인...
JiHyeonL pushed 1 commit to hoeseong123 woowacourse/java-coupon
- [DB 복제와 캐시 1단계] 초롱 미션 제출합니다. (#36) * docs: 기능 요구사항 작성 * feat: 쿠폰 생성 및 이름 검증 기능 구현 * feat: 쿠폰 할인 금액 검증 구현 * ... 98712bc
JiHyeonL closed a pull request on woowacourse/java-coupon
[DB 복제와 캐시 1단계] 초롱 미션 제출합니다.
안녕하세요 레모네!!!!🍋 제가 생각한 복제 지연에 대한 해결 방법은 다음과 같아요. - 데이터 읽기에 실패할 경우 Writer DB에서 재시도 한다. 이렇게 했을 때 생각나는 문제점은 다음과 같아요. - 악의적인 사용자가 존재하지 않는 데이터에 대한 읽기 요청을 무한하게 보낼 경우 심각한 성능 저하가 발생할 수 있습니다. - upda...
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
클래스 선언부 다음으로 오는 필드 사이에 공백을 추가한 클래스도 있고, 공백이 없는 클래스도 있어서 컨벤션을 맞춰주시면 좋을 것 같아요!
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
Member와 MemberCoupon 클래스 마지막에 불필요한 공백 삭제해 주세요!
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
위에서 언급한대로 id 파라미터가 없는 생성자가 있다면 이 생성자도 불필요 할 것 같아요.
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
생성자 파라미터로 id가 존재하면, 개발자가 임의로 id를 조작할 수 있다는 문제점이 발생할 수 있겠네요! Id를 조작할 수 있는 권한은 JPA에게만 부여하는 것이 더 안정적일 것 같아요.
JiHyeonL created a review comment on a pull request on woowacourse/java-coupon
두 값은 null을 허용하지 않는데 래퍼 클래스를 사용하신 이유가 있을까요?
JiHyeonL created a review on a pull request on woowacourse/java-coupon
안녕하세요 초롱! 1단계 미션 요구사항은 모두 만족하신 것 같아 approve 드리겠습니다. 추가적으로 도메인 & 클린코드 관련 피드백을 남겼는데요, 2단계 진행하시면서 함께 반영해 주시면 좋을 것 같아요!! 수고하셨습니다~~
JiHyeonL created a review on a pull request on woowacourse/java-coupon
안녕하세요 초롱! 1단계 미션 요구사항은 모두 만족하신 것 같아 approve 드리겠습니다. 추가적으로 도메인 & 클린코드 관련 피드백을 남겼는데요, 2단계 진행하시면서 함께 반영해 주시면 좋을 것 같아요!! 수고하셨습니다~~
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
복제 지연 없는 복제 지연 테스트였네요 이럴수가... 반영했습니다!
mzeong created a review comment on a pull request on woowacourse/java-coupon
저는 회원의 쿠폰 목록 조회 기능이 추가되면 TTL을 좀 늘려야 하나? 라는 고민을 들었는데 명오는 어떠셨나요?? 설정을 유지한 명오의 이유가 궁금해요!
mzeong created a review comment on a pull request on woowacourse/java-coupon
해당 캐시 전략을 선택한 이유가 무엇인가요?? 👀
mzeong created a review comment on a pull request on woowacourse/java-coupon
member coupon에 관해서는 캐싱을 안 하고 있는 것 같은데 초기화 코드를 작성해준 것은 추후를 고려한 건가요?
mzeong created a review on a pull request on woowacourse/java-coupon
안녕하세요 명오! 제리입니다 이미 캐시를 구현해두신 상태라 간단한 리뷰만 남겼습니다 👍 최종 데모데이 준비하시느라 고생 많으셨습니다 즐거운 주말 보내세요~!
mzeong created a review on a pull request on woowacourse/java-coupon
안녕하세요 명오! 제리입니다 이미 캐시를 구현해두신 상태라 간단한 리뷰만 남겼습니다 👍 최종 데모데이 준비하시느라 고생 많으셨습니다 즐거운 주말 보내세요~!
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
하나의 서비스 객체에서 트랜잭션 옵션을 관리하는 방식도 고려해 보았으나, 이렇게 할 경우 서비스가 트랜잭션 관리를 깊이 담당하게 되어 같은 맥락의 코드를 DataSource만 변경하여 조회하는 것이 적절한 역할 분배인가? 생각이 들었어요 반면, 읽기와 쓰기 서비스를 분리하면 각 서비스가 자체적으로 데이터 소스를 인식하는 것이 전자의 코드보다는 ...
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
어디 가서 우테코 출신이라 말할 수 있게 분발하겠습니다
shin-jisong created a review comment on a pull request on woowacourse/java-coupon
오 흥미로운 구현이네요! 이러한 구현 방식도 있구나 참고하였습니다