INTERNAL: add builder and transaction for ArcusCacheManager

[oliviarla]

🔗 Related Issue

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

⌨️ What I did

• 현재 대부분의 CacheManager 구현체가 상속받고 있는 AbstractTransactionSupportingCacheManager를 상속받도록 변경했습니다. 이에 따라 spring-context-support, spring-tx 의존성이 추가되었습니다.
• CacheManager 빌더를 추가했습니다.
• ArcusCacheConfiguration에 default config를 얻을 수 있는 메서드를 추가했습니다. 이를 통해 빌더를 사용하면 반드시 default config를 입력하지 않아도 됩니다.

[jhpark816]

@oliviarla
질문입니다.
AbstractTransactionSupportingCacheManager를 상속받아 얻는 이득은 무엇인가요?

[oliviarla]

@jhpark816

AbstractTransactionSupportingCacheManager를 상속받아 얻는 이득은 무엇인가요?

Spring Transaction에 의해 관리될 수 있도록 하는 것이 목적입니다. ArcusCache의 put, evict, clear 메서드가 호출되었을 때 Spring 트랜잭션의 커밋 후에 수행되도록 합니다.

참고로 Spring Transaction의 커밋 시점에 맞추어 작업이 수행되도록 지정할 수 있는데요, beforeCommit, beforeCompletion, afterCommit(AbstractTransactionSupportingCacheManager를 통해 적용하는 시점), afterCompletion 네 가지 시점에 수행되도록 설정할 수 있습니다.

현재 타사 Cache 구현체들에서는 대부분 이를 상속받고 있습니다.

[jhpark816]

일부 리뷰

[jhpark816]

아래 두 메소드는 없어도 될 것 같은데요. 필요한 데가 있나요?

• getCacheConfigurationFor()
• getConfiguredCaches()

[jhpark816]

메소드 명 : defaultCacheConfiguration() or defaultConfiguration() ??

    public ArcusCacheManagerBuilder defaultCacheConfiguration(ArcusCacheConfiguration cacheConfiguration) {
      this.defaultConfiguration = cacheConfiguration;
      return this;
    }

    public ArcusCacheManagerBuilder defaultConfiguration(ArcusCacheConfiguration configuration) {
      this.defaultConfiguration = configuration;
      return this;
    }

[jhpark816]

• initialCacheNames() vs initialCaches()

아래의 다른 메소드까지 고려한다면

• withInitialCachesWithDefault()
• withInitialCacheWithConfig()
• withInitialCachesWithConfig()

이름을 만드는 것이 힘드네요.

[jhpark816]

아래 코드 형태를 생각해 보았을 것으로 생각합니다.
하지만, 위의 코드를 넣은 것은 cacheDefaults()의 다양한 활용 예를 보이려는 의도인가요?

    ArcusCacheManager cm = ArcusCacheManager.builder(arcusClientPool)
            .withCacheConfiguration("first-cache", withPrefix)
            .withCacheConfiguration("second-cache", withoutPrefix)
            .build();

[jhpark816]

setServiceId() 수행하지 않으면 어떻게 되나요?

[jhpark816]

@oliviarla
아래와 같이 정리합시다.

• ServiceId 디폴트 설정을 미리 넣어둡시다.
• Builder 메소드 용어는 현재 PR대로 합시다.