Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ChooSeoyeon created a review comment on a pull request on woowacourse/java-coupon
급하게 짜다 이런 큰 실수를 범했네요.... 애매하게 알고 있던 개념이라 이번 기회에 정리하고 넘어가겠슴다 ✅ [fix: 프록시 기반 AOP 정상동작하도록 수정](https://github.com/woowacourse/java-coupon/pull/57/commits/e098fa0826947ce39a8360f00e2b29a68fc99bea) ...
ikjo39 created a review on a pull request on woowacourse/java-coupon
안녕하세요 배키 다온입니다 👋 남겨주신 답변 모두 확인하였습니다. 열심히 고민하신 흔적이 느껴지네요. 이번 단계에서 충분히 얘기나눴다 생각하여 이만 머지할게요. 다음 단계에서 만나요 💪
ikjo39 created a review comment on a pull request on woowacourse/java-coupon
제가 의도한 방향은 JPA는 쿼리로 가져온 데이터를 필드에 Reflection을 이용하여 직접 넣어주기 때문에 테스트가 통과됨을 의도하였습니다 :) 제가 애매하게 질문 드린 것 같아요. 배키가 해주신 말씀도 제 질문에 답변이 될 수 있을 거 같네요 ㅎ
ikjo39 created a review comment on a pull request on woowacourse/java-coupon
이 부분은 아무설정이 없을 경우 SpringBoot가 기본적으로 적용하는 DataSource 구현체를 여쭤본것이었답니다. 😁
ikjo39 created a review comment on a pull request on woowacourse/java-coupon
현재는 관리 주체가 메서드명과, Qualifier에 해당하는 상수명으로 2개가 되버린듯해요. 제가 의도한 방향은 `@Qualifier`가 필요한 빈에 대해서만 명시적으로 빈 이름을 등록하길 바랬습니다. (`readDataSource`와 `writeDataSource`) 배키 말씀처럼 기본적으로 메서드명을 이용하여 빈 등록이 되니까요. 😁 확...
ikjo39 created a review comment on a pull request on woowacourse/java-coupon
죠습니다! 추가적으로 `StringUtils.isBlank()`를 사용하시면 Null 까지 검증됩니다!
ksk0605 pushed 1 commit to fromitive woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 포케(이무송) 미션 제출합니다. (#20) * feat: 이름 기능 구현 * feat: 할인 금액 기능 구현 * chore: assertJ 적용 * feat: ... de07e65
ksk0605 closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 포케(이무송) 미션 제출합니다.
안녕하세요 테바! 😄 포케 입니다. 이번 미션은 복제 지연을 어떻게 해결할지 고민하고 의견을 내는 것을 목표로 두었습니다. 복제 지연을 해결하기 위해 대략 아래의 방법들을 적용하면 해결 가능하다고 생각합니다. ``` 1. 상대적으로 빠른 캐시 메모리에 생성한 쿠폰 정보를 저장하고. 캐시에 우선 조회한다. 2. writer DB에 ...
ksk0605 created a review on a pull request on woowacourse/java-coupon
안녕하세요 포케 많이 늦어서 미안해요..😥 그 사이에 코드에는 많은 변경사항이 있었네요. 캐시를 사용하게 되면서 또 다시 고민해볼 거리가 생겼어요. 남겨놓은 코멘트는 다음번 제출에 답변 부탁드려요 😀 이번 미션은 충분하다고 생각해서 이만 머지하겠습니다. 다음 스탭에서 뵈어요!
ksk0605 created a review on a pull request on woowacourse/java-coupon
안녕하세요 포케 많이 늦어서 미안해요..😥 그 사이에 코드에는 많은 변경사항이 있었네요. 캐시를 사용하게 되면서 또 다시 고민해볼 거리가 생겼어요. 남겨놓은 코멘트는 다음번 제출에 답변 부탁드려요 😀 이번 미션은 충분하다고 생각해서 이만 머지하겠습니다. 다음 스탭에서 뵈어요!
ehBeak created a comment on a pull request on woowacourse/java-coupon
안녕하세요 :) 다온👋 정말 많이 늦었네요.. 하하 죄송합니다. 꼼꼼한 리뷰 내용 정말 감사해요. 덕분에 저도 알아보고 좋은 시간이었습니다. --- > 다만 복제 지연을 해결하기 위한 다양한 조치들이 있는데 왜 이러한 방식을 선택했는지, 어떤 게 좋고, 어떤 게 문제가 될 수 있을지와 같은 근거가 부족하다 느꼈어요. > 면접관은 ...
BurningFalls opened a pull request on woowacourse/java-coupon
[2단계 - DB 복제와 캐시] 폭포(이성주) 미션 제출합니다.
안녕하세요, 호티! 2단계 DB 복제와 캐시 구현 완료하여 PR 보냅니다. 1단계에서 구현했던대로 로컬 캐시를 사용해서 두 기능을 구현했습니다. * 회원에게 쿠폰을 발급하는 기능 * 회원의 쿠폰 목록 조회 기능 create 할때는 캐시와 실제 DB의 데이터 정합성이 보장되어야 하기 때문에, 항상 메서드를 실행시키고 캐시 값도 갱신하는...
geoje pushed 1 commit to hjk0761 woowacourse/java-coupon
- [1단계 - 복제 지연] 무빈(김현중) 미션 제출합니다. (#39) * feat: 도메인 생성 * feat: 쿠폰 서비스 및 레포지토리 생성 * feat: DataSource 설정 구현 * feat: d... e54ed96
geoje closed a pull request on woowacourse/java-coupon
[1단계 - 복제 지연] 무빈(김현중) 미션 제출합니다.
새양 안녕하세요!! 미션으로는 처음 만나는 것 같아 반갑네요 😄 ### 이번 단계에서 구현한 것 - 쿠폰 도메인을 구현하고, 조건에 맞는 검증 로직과 테스트를 구현했습니다. - 복제 지연을 확인하고, 이를 해결하기 위해 DataSource 를 찾는 작업을 추가했습니다. 이번 단계의 요구사항을 충족하는 것을 최우선으로 했습니다. ...
geoje created a review on a pull request on woowacourse/java-coupon
안녕하세요 무빈~ 반영 정말 잘 해주시고 답변도 잘 해주셨네요!! 이번 단계 학습 사항 충분히 만족 하신 것 같아 이만 머지하겠습니다! 다음 단계에서 뵈요~ 파이팅입니다!! 🎉
geoje created a review comment on a pull request on woowacourse/java-coupon
헉 ㅋㅋㅋ 저는 의견을 나누고 싶었는데 반영을 바로 해버리셨군요... 저는 오히려 메서드명이나 어노테이션의 설명 등으로 숫자의 의미가 파악이 된다면은 굳이 매직넘버로 보고 분리해야하나 라는 입장이었습니다 🥲
jcoding-play pushed 1 commit to kimprodp woowacourse/java-coupon
- [1단계 - DB 복제와 캐시] 테드(김규태) 미션 제출합니다. (#61) * chore: ddl auto update * chore: schema.sql 작성 * feat: coupon 구현 ... cce1125
jcoding-play closed a pull request on woowacourse/java-coupon
[1단계 - DB 복제와 캐시] 테드(김규태) 미션 제출합니다.
안녕하세요 뽀로로~ 오랜만입니다. 잘 지내시죠? 😁 복제와 캐시 미션 1단계 완료하여 PR 요청 드립니다. DB 설정과 복제가 제대로 되지 않아서 삽질 좀 하다가 늦어졌네요.. 복제 지연에 대한 해결책으로 저는 단순하게 **reader 에서 조회할 경우 writer를 확인하여 가져오는 방법**을 선택했습니다. 어떤 방법으로 할까...
jcoding-play created a review on a pull request on woowacourse/java-coupon
안녕하세요~ 테드! 수정사항 확인했고, 이번 단계는 머지하도록 할게요! 수고하셨습니다! 리뷰 요청 확인이 늦어 죄송해요! 😭
BurningFalls created a review comment on a pull request on woowacourse/java-coupon
발급일 포함 7일동안 사용 가능이기 때문에 6을 더해줘야 하네요! 놓쳤던 부분입니다. 수정 완료했습니다.
geoje created a review comment on a pull request on woowacourse/java-coupon
중복 사용에 대한 대답을 유도한 것인데 정확히 캐치해주셨군요!! 👍
geoje created a review comment on a pull request on woowacourse/java-coupon
정확히 파악해주셨습니다!! 저희가 AOP 를 통해 트랜잭션 구현해 보았기에 이 객체 또한 작동 원리를 파악하셨을거라 생각해요! 👍