HONG CHANGSUB
HONG CHANGSUB
@uhm0311 리뷰 부탁드립니다.
@jhpark816 최종 리뷰 부탁드립니다.
@uhm0311 ArcusClient만 사용하는 경우 사용자가 커스텀하게 name을 주는 경우는 client_index가 적용되지 않도록 변경하였습니다.
@jhpark816 ArcusClient 객체 생성의 책임을 ArcusClientPool에게 옮기는 작업은 @uhm0311 님과 논의했을 때도 CacheManager의 책임을 줄이기에 좋은 방향이라는 것에 서로 동의하였습니다. 다만, 제가 판단하건데 이러한 변경 사항이 본 PR과는 별개로 처리되어야...
https://github.com/jam2in/arcus-works/issues/549 현재 PR에서는 locator의 update와 handleDelayedClosingNodes 모두 IO 쓰레드에서 순차적으로 수행합니다. 만약, 현재 진행중인 이슈처럼 locator에 대한 update 수행을 다른 스레드가 수행할 경우 해당 구현도 변경이 필요할 수도 있습니다. 추후...
> 동시성 이슈는 어떻게 해결했나요? `ArcusKetamaNodeLocator` 내부에 존재하는 Lock 인스턴스를 통해 `allNodes` 접근하는 경우 lock을 걸어 동시성 이슈를 해결하려 합니다. 이렇게 될 경우 IO Thread가 getAllNodes()를 호출할 때 Blocking이 될...
HashRingUpdate Task의 작업 결과를 MemcachedConnection의 Future 필드로 가질 수 있게 하였습니다. 결론적으로 테스트 코드에서 `getHashUpdateResult()`를 호출해 Hash Ring update가 정상적으로 이루어졌는지 boolean 값을 통해 확인 할 수 있습니다.
@jhpark816 리뷰 부탁드리겠습니다.
@uhm0311 @oliviarla @jhpark816 `handleCacheNodesChange` 메서드 로직 전체를 별도의 스레드가 수행하도록 변경하였습니다.
@uhm0311 @oliviarla 리뷰가 완료되신걸까요?