search

naver / arcus-spring

ARCUS as a caching provider for the Spring Cache Abstraction
Apache License 2.0
26 stars 16 forks source link

createArcusKey()에서 arcusKey 생성 부분 수정. #24

Closed jhpark816 closed 3 years ago

jhpark816 commented 3 years ago

현재 createArcusKey() 메소드에서 arcusKey 생성 부분이 아래와 같습니다.

  public String createArcusKey(final Object key) {
    String keyString, arcusKey;

    . . .
    arcusKey = serviceId + name + ":" + keyString;
    if (this.prefix != null) {
      arcusKey = serviceId + prefix + ":" + keyString;
    }
    . . .

    return arcusKey;
  }

위의 코드는 불필요하게 GC 부담을 가중시킬 수 있으므로, 아래와 변경하는 것이 좋겠습니다.

    if (this.prefix != null) {
      arcusKey = serviceId + prefix + ":" + keyString;
    } else {
      arcusKey = serviceId + name + ":" + keyString;
    }
hjyun328 commented 3 years ago

PR merge되어 클로즈합니다.