INTERNAL: provide meaningful methods instead of setters in ArcusCacheConfiguration

[oliviarla]

🔗 Related Issue

⌨️ What I did

• ArcusCacheConfiguration으로 사용자가 값을 설정할 때 setter 대신 withServiceId, enableFrontCaching 등의 메서드를 사용하도록 변경하고 해당 메서드 안에서 입력값을 검증하도록 합니다.
• 사용자 입장에서 코드가 좀 더 간결해진다는 장점이 있으며, Redis와 Couchbase와 비슷한 방식으로 사용할 수 있습니다.
• setter를 deprecate시키기 때문에, 나중에 setter가 제거되면 xml으로 ArcusCacheConfiguration의 각 필드를 설정할 수 없게 됩니다. 반드시 xml 빈 등록 방식을 지원해야 한다면 ArcusCacheConfigurationFactoryBean 클래스를 만들고 이 안에 setter를 두면 xml기반 설정 방식을 지원할 수 있습니다. 의견 부탁드립니다.
• 기존 ArcusCache에 있던 기본값들을 ArcusCacheConfiguration에서 더 많이 사용하므로 옮겼습니다.

[jhpark816]

일부 리뷰

[oliviarla]

@brido4125 @uhm0311
기존 방식에서 현재 방식으로 변경하는 것이 괜찮은지 의견 부탁드립니다.

[brido4125]

저는 좋다고 생각되어서 리뷰 진행하였습니다.

[brido4125]

기존 사용자가 ArcusCacheConfiguration을 InitializingBean 타입으로 사용한 로직은
거의 없겠지만 타입이 변경되는 거니 릴리즈 할 때 별도의 알림은 필요할 것 같습니다.

[jhpark816]

@oliviarla

아래 코드에 있는 config 객체 생성 방식도 참고 바랍니다.

  public LocalCacheManager(String name, int max, int exptime, boolean copyOnRead,
                           boolean copyOnWrite) {
    this.cache = CacheManager.getInstance().getCache(name);
    if (cache == null) {
      CacheConfiguration config =
              new CacheConfiguration(name, max)
                      .copyOnRead(copyOnRead)
                      .copyOnWrite(copyOnWrite)
                      .memoryStoreEvictionPolicy(MemoryStoreEvictionPolicy.LRU)
                      .eternal(false)
                      .timeToLiveSeconds(exptime)
                      .timeToIdleSeconds(exptime)
                      .diskExpiryThreadIntervalSeconds(60)
                      .persistence(new PersistenceConfiguration().strategy(
                          PersistenceConfiguration.Strategy.NONE));
      this.cache = new Cache(config, null, null);
      CacheManager.getInstance().addCache(cache);

      logger.info("Arcus k/v local cache is enabled : %s", cache.toString());
    }
  }

[jhpark816]

리뷰 완료

[jhpark816]

리뷰 완료

[jhpark816]

조회 요청이 아닌 set, .... 등의 변경 요청에 대한 설정이며,
변경 요청이 실패하더라도 Front cache에 set, ... 등으로 저장해 둘 것인지를 설정하는 것입니다.

이러한 내용이 명확하게 드러나면 좋겠습니다.

[oliviarla]

반영했습니다.