search

jam2in / arcus-java-client

Arcus Java client
Apache License 2.0
0 stars 0 forks source link

Refactoring seperate CacheMonitor Zookeeper watcher from CacheManager #38

Closed whchoi83 closed 7 years ago

whchoi83 commented 7 years ago

CacheManager 를 default watcher 로 등록하고 모든 event 를 받아 CacheMonitor 로 전달하던 것에서 CacheManager 와 CacheMonitor 의 역할에 맞게 서로 분리했습니다.

차후에 MigrationMonitor(미정) 를 추가하기 위해서라 default watcher 를 사용하던 것에서 목적에 맞게 watcher 를 분리하는 작업이 필요합니다

본 PR은 리뷰가 끝난 뒤에 Mingration 개발이 끝나고 PR & Merge 시점에 함께 merge 하는 것이 좋을 것 같습니다. 추가적인 테스트와 migration 개발 중 변경 가능성이 있어서 입니다.

whchoi83 commented 7 years ago

공식적인 리뷰어를 @aiceru 에서 @MinWooJin @minkikim89 로 변경했습니다 @aiceru 님도 기본적인 리뷰를 해주시면 좋을 것 같습니다. 😄

aiceru commented 7 years ago

Review 완료. 이상없어보입니다~

MinWooJin commented 7 years ago

리뷰 완료 했습니다~

minkikim89 commented 7 years ago

리뷰 완료했습니다~