arcus-java-client
arcus-java-client copied to clipboard
FEAT: Change the way Arcusclient is named.
관련 이슈
https://github.com/jam2in/arcus-works/issues/344, https://github.com/jam2in/arcus-works/issues/490
변경 내용
ArcusClientPool 사용 시
ArcusClient-1(1-4)와 같은 네이밍으로 변경 ArcusClient의 첫번째 풀이고, 4개의 사이즈 중 1번째 클라이언트 인스턴스임을 의미
ArcusClient만 사용시
- 사용자가 name을 생성자의 인자로 줄 경우 ex) MyClient3 -> MyClient는 사용자가 생성자의 인자로 넘겨주었고, 3은 ArcusClient객체만 사용될 경우 몇번 째 인스턴스인지 의미함
- default 설정된 name 사용 시 ex) ArucsClient2 -> "ArucsClient"는 디폴트 네임이고, ArcusClient객체만 사용하는 2번째 인스턴스라는 의미
rebase 해주세요.
@uhm0311 ArcusClient만 사용하는 경우 사용자가 커스텀하게 name을 주는 경우는 client_index가 적용되지 않도록 변경하였습니다.
@jhpark816
ArcusClient 객체 생성의 책임을 ArcusClientPool에게 옮기는 작업은 @uhm0311 님과 논의했을 때도 CacheManager의 책임을 줄이기에 좋은 방향이라는 것에 서로 동의하였습니다.
다만, 제가 판단하건데 이러한 변경 사항이 본 PR과는 별개로 처리되어야 할 것 같습니다. CacheManager의 어떤 책임을 분리시키고 어떤식으로 구현할지에 대해 신중하게 설계를 해야하기 때문입니다. 또한 본 PR의 목적과도 전혀 다르기 때문입니다.
말씀하신 부분은 새로운 이슈로 생성하고 별도로 처리하도록 하겠습니다.
@uhm0311 최종 리뷰 부탁합니다.