FEAT: Add gat(get and touch) command to get item & update item expira…

[singsangssong]

🔗 Related Issue

• touch API 개발 #295
• touch 기능 개발 arcus-memcached#487

⌨️ What I did

memcached 서버의 gat를 java-client에도 추가합니다.
이를 통해 expire time을 재설정 가능하도록 합니다.

• Operation 인터페이스 및 Ascii, Binary 프로토콜 구현체 추가
• OperationFactory에 GetAndTouch 추가
• MemcachedClient에 사용자가 호출할 수 있는 gat 기능 추가
• gat 기능을 위한 테스트 코드 추가

CI Test 실패 관련

현재 실패하고 있는 CI Test는 JDK 21, ubuntu-latest에서 Test ARCUS Client with ZK를 실패하고 있습니다.
코드에서는 AsciiClientTest의 gat관련 테스트를 실패하고 있습니다.
해당 테스트는 현재 도커에 릴리즈된 memcached 서버는 gat 기능이 아직 반영되지 않았기에, 테스트코드에서 gat명령을 요청해도 응답이 없는 문제입니다.

[uhm0311]

아스키 명령어는 gat를 호출하고 있고, API 구현은 gats를 호출하는 것 기준으로 되어 있습니다.
CASValue 타입을 반환하지 않도록 수정하거나, gats 아스키 명령어를 호출하도록 변경해야 할 것 같습니다.

[singsangssong]

아스키 명령어는 gat를 호출하고 있고, API 구현은 gats를 호출하는 것 기준으로 되어 있습니다.
CASValue 타입을 반환하지 않도록 수정하거나, gats 아스키 명령어를 호출하도록 변경해야 할 것 같습니다.

네 알겠습니다. CASValue 타입을 반환하지 않도록 수정한 후, 아스키 명령어는 gat를 호출하도록 코드를 수정하겠습니다.

[uhm0311]

커밋 개수가 갑자기 많아졌는데, 어떻게 된 일인가요?

[singsangssong]

develop 브랜치의 최신 변경사항을 제 작업 브랜치에 반영하는 과정에서, rebase 대신 merge를 사용하여 develop 브랜치의 모든 커밋들이 함께 포함되었고, PR의 대상브랜치가 master로 변경된 점이 문제가 되었습니다.

앞으로 PR작업 주의하며 수행하겠습니다.

[uhm0311]

@oliviarla

의견이 궁금해서 질문합니다.

gat의 응답은 get 명령어의 응답과 동일한 포맷입니다.
따라서 GetAndTouchOperation이 GetOperation을 상속받게 만들어도 될 것 같고, API의 반환 타입도 GetFuture로 해도 될 것 같습니다.
GetAndTouchOperation이 GetOperation을 상속받은 다음, 아스키 명령어를 초기화하는 메소드를 오버라이드하면 구현이 더욱 간편해집니다.
어떻게 생각하시나요?

[oliviarla]

@uhm0311

touch, gat 명령을 추가한 것은 memcached 진영과의 호환을 위한 것인데, 현재 spymemcached의 ascii API에서는 touch 만 지원하고 gat은 지원하지 않습니다.(UnsupportedException 발생) 따라서 gat, gats API는 저희가 자체적으로 정의하여 제공하면 될 것 같습니다. 즉, 말씀하신대로 GetFuture를 반환하도록 해도 인터페이스 호환 상 문제가 없습니다.

내부 Operation 생성 관점에서는 중복을 최소화하는 방향으로 가는 것에 동의합니다. 그러면 GetAndTouchOperationImpl, GetsAndTouchOperationImpl 클래스를 새로 생성하여 BaseGetOperationImpl을 상속받도록 하고, 각각 GetOperation, GetsOperation을 구현하게 되나요?

[uhm0311]

BaseGetOpImpl 클래스를 상속하게 하고, BaseGetOpImpl 클래스에 gat 명령어를 위한 expTime 필드를 추가한 다음, BaseGetOpImpl 클래스의 일부 메소드를 조금 수정하면 될 것 같습니다.

[singsangssong]

처음 구현할 때,BaseGetOpImpl클래스에 expTime 필드가 없는 점과, 메서드가 GetAndTouchOperationImpl 클래스를 상속하기 힘들다고 판단했습니다.

코멘트주신 사항을 아래 정리했습니다. 이 내용을 바탕으로 코드를 수정하겠습니다.

• GetAndTouchOperationImpl 클래스가 BaseGetOpImpl 클래스를 상속 (이때, GetAndTouch에 맞는 메서드 및 필드 수정)
• API 반환 타입 GetFuture 반환
• GetsAndTouchOperationImpl클래스 생성

[oliviarla]

@jhpark816
getAndTouch는 spymemcached와 호환되지 않기 때문에, asyncGetAndTouch 메서드와 asyncGetsAndTouch 메서드만 제공하면 될 것 같은데 어떻게 생각하시나요?

[jhpark816]

asyncGetAndTouch 메서드와 asyncGetsAndTouch 메서드만 제공

위와 같이 진행하면 됩니다.

[singsangssong]

@uhm0311 @oliviarla
리뷰 부탁드립니다. 상단에 코멘트주신대로 asyncGetAndTouch, asyncGetsAndTouch 메서드만 구현완료했습니다.

[oliviarla]

리뷰 완료입니다.

[jhpark816]

리뷰 완료

[jhpark816]

GET, GETS에 대응되게 GAT 외에도 GATS를 둡시다.

[jhpark816]

OperationType.java 파일에서
Operation Hierarchy 부분에도 반영 바랍니다.

[singsangssong]

코멘트 내용 모두 반영했습니다.
그 외에 BaseGetOpImpl에서 Replication을 위해 코드가 추가되었으니, 확인 후 approve 부탁드립니다.

// BaseGetOpImpl.java
@Override
  public final void handleLine(String line) {
    /*
      VALUE <key> <flags> <bytes> [<cas unique>]\r\n
      <data block>\r\n
      ...
      END\r\n
    */
    /* ENABLE_REPLICATION if */
    if (hasSwitchedOver(line)) {
      prepareSwitchover(line);
      return;
    }
    /* ENABLE_REPLICATION end */
    if (line.equals("END")) {
    ....

[jhpark816]

리뷰 완료

[jhpark816]

아래와 같이 정리되어야 할 것 같습니다.

GetAndTouchOperationImpl / GetsAndTouchOperationImpl (asyncGetAndTouch / asyncGetsAndTouch)

[jhpark816]

gat, gats에서 exp 값을 0을 줄 수 있습니다.
따라서, exp 값 자체만으로는 정상적인 값인지 확인할 수 없고,
cmd 값을 확인하고 exp 사용 여부를 결정해야 합니다.
그러면, 여기서 exp 값을 설정해 둘 필요가 있는 지 모르겠어요.

[jhpark816]

@uhm0311 @oliviarla

#941 (comment) 코멘트를 확인하세요.

[oliviarla]

@jhpark816
gat/gats 명령에서 replication, migration 관련 응답을 보내지 않으므로 해당 처리는 제외하여도 됩니다.

[jhpark816]

@ing-eoking
캐시 서버에서 아래 부분을 확인해 주세요.

gat/gats 명령에서 replication, migration 관련 응답을 보내지 않으므로 해당 처리는 제외하여도 됩니다.

[ing-eoking]

@jhpark816

캐시 서버에서 아래 부분을 확인해 주세요.

네, 캐시 서버에서는 get and touch 연산 중 실패가 발생하더라도 별도의 에러 응답을 보내지 않습니다.

• FEATURE: Add gat command arcus-memcached#781 (comment)

gat 명령어는 내부적으로 get -> touch(setattr) 순서로 동작하며,
CLIENT_ERROR나 SERVER_ERROR와 같은 예외적인 경우를 제외하면, 다음과 같은 상황에서 실패할 수 있습니다.

• get 과정에서 실패한 경우 (예: 캐시 미스, 타입 불일치)
• get은 성공했지만, 이후 touch(setattr) 과정에서 실패한 경우

이 중 replication이나 migration 관련 이슈로 인한 실패는 두 번째 경우에 해당하며,
이 경우에도 서버는 해당 키에 대해 별도의 응답을 보내지 않습니다.

이는 gat 연산이 기존 get 연산의 로직을 그대로 활용하면서 유사한 방식으로 동작하도록 구현되었기 때문입니다.
기존의 get 연산 또한 실패하더라도 별도의 에러 메시지를 구성하지 않고, 해당 키를 응답에서 제외하는 방식으로 처리됩니다.

즉, 기존 get 연산 역시 해당 아이템이 캐시 미스로 실패했는지, 타입 불일치로 실패했는지에 대한 구분 없이 처리됩니다.
gat 연산도 마찬가지로, 연산 과정에서 에러가 발생하더라도 해당 아이템에 대해서는 단순히 결과에서 제외하는 방식으로 구현된 것으로 기억합니다.