FEATURE: Add DefaultArcusConnectionFactory class.

[uhm0311]

🔗 Related Issue

• https://github.com/jam2in/arcus-works/issues/570

⌨️ What I did

ArcusConnectionFactory 클래스를 추가합니다. 대부분의 구현이 DefaultConnectionFactory와 같아 DefaultConnectionFactory를 상속받도록 했습니다.

static final 필드 추가

다음과 같은 기준으로 ConnectionFactoryBuilder 클래스의 필드 기본값을 ArcusConnectionFactory 클래스 내에 static final로 선언했습니다.

• DefaultConnectionFactory 클래스에 이미 static final로 선언되어 있지만 ConnectionFactoryBuilder 클래스의 필드 기본값과 다른 경우
  • DEFAULT_FAILURE_MODE = FailureMode.Cancel
  • DEFAULT_IS_DAEMON = true
  • DEFAULT_MAX_RECONNECT_DELAY = 1
  • HashAlgorithm DEFAULT_HASH = HashAlgorithm.KETAMA_HASH
  • DEFAULT_MAX_TIMEOUTEXCEPTION_THRESHOLD = 10
  • DEFAULT_MAX_TIMEOUTDURATION_THRESHOLD = 1000

기본값을 리터럴에서 static final 필드로 변경

ConnectionFactoryBuilder 클래스의 필드 기본값 중 리터럴로 하드코딩 되어 있는 경우 내부 동작이 달라지지 않는 선에서 static final 값을 참조하도록 했습니다. 만약 필드의 기본값으로 할당할 static final 필드가 없다면 새롭게 추가했습니다. 단, 아래와 같은 경우 기존대로 리터럴을 기본값으로 사용하도록 했습니다.

• ARCUS 캐시 클러스터가 기준임을 명확히 알 수 있는 경우
  • private Locator locator = Locator.ARCUSCONSISTENT;
  • Replication, Migration 관련 필드들
• 기본값이 명확하며, 서로 다른 세 가지 방식으로 ConnectionFactory 객체를 생성해도 기본값이 같은 경우
  • private Collection<ConnectionObserver> initialObservers = Collections.emptyList();
  • private AuthDescriptor authDescriptor = null;

테스트 코드

ArcusConnectionFactory와 다음의 두 가지를 비교하는 테스트 코드를 추가했습니다.

• DefaultConnectionFactory
• ConnectionFactoryBuilder

향후 기본값을 잘못 변경했을 때 테스트 코드를 통해 알아차릴 수 있도록 Reflection을 이용하여 서로 다른 결과를 반환해야 하는 메소드가 아니라면 동일한 값이 반환되는지 검증하기 때문에 테스트 코드가 엄격해 보일 수 있습니다. 단, 메소드가 Primitive Type도 아니고 String 타입도 아닌 값을 반환하는 경우 Object.eqauls() 메소드를 구현하지 않은 객체는 값을 비교할 수 없어 반환된 객체의 값이 아닌 타입이 동일한지 검증합니다.

[jhpark816]

@uhm0311 아래와 같이 wiki에 ConnectionFactory를 충분히 설명하였고, ConnectionFactoryBuider로 원하는 ConnectionFactory 만들 수 있으므로, DefaultArcusConnectionFactory는 추가하지 않는 것으로 하겠습니다.

• https://github.com/naver/arcus-java-client/wiki/DefaultConnectionFactory-%ED%81%B4%EB%9E%98%EC%8A%A4%EC%99%80-ConnectionFactoryBuilder-%ED%81%B4%EB%9E%98%EC%8A%A4%EC%9D%98-%EC%B0%A8%EC%9D%B4%EC%A0%90