INTERNAL: Remove public ArcusClient constructor.

naver / arcus-java-client

ARCUS Java client

Apache License 2.0

50 stars 47 forks source link

Closed brido4125 closed 3 weeks ago

brido4125 commented 5 months ago

~~public으로 설정된 ArcusClient 생성자를 외부 사용자가 접근 할 수 없게 protected로 변경했습니다. 추후 ArcusClient가 상속 구조를 가질 수도 있을 것 같아 private으로 설정하지 않았습니다.~~

블라블라..

이와 더불어 기존에 memcached 주소로 연결할 수 있는 기능은 createArcusClient or createArcusClientPool 이라는 스태틱 메서드로 제공합니다. (pool도 제공)

jhpark816 commented 5 months ago

@oliviarla 추가 리뷰어 지정합니다.

uhm0311 commented 1 month ago

PR 제목을 실제 변경사항과 일치하게 바꿔주세요.

jhpark816 commented 1 month ago

추후 ArcusClient가 상속 구조를 가질 수도 있을 것 같아 private으로 설정하지 않았습니다.

ArcusClient 상속할 경우가 추후에 발생하나요?

brido4125 commented 1 month ago

ArcusClient 상속할 경우가 추후에 발생하나요?

FrontCacheMemcachedClient와 MemcachedClient의 관계처럼 ArcusClient도 플러그인 성격의 어떠한 클래스를 상속할 수도 있다고 생각합니다.

jhpark816 commented 3 weeks ago

그리고, PR 처음 코멘트는 현재 수정 사항을 나타내도록 변경되어야 합니다.

brido4125 commented 3 weeks ago

@jhpark816 님 의견에 따라 close 합니다.