fix: use server metadata for item not found (#369)

Consumes the server metadata change from element_not_found to item_not_found.
momentohq · Jun 24, 2024 · a6b9f09 · a6b9f09
1 parent d5896ff
commit a6b9f09
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/momento-sdk/src/main/java/momento/sdk/exceptions/CacheServiceExceptionMapper.java b/momento-sdk/src/main/java/momento/sdk/exceptions/CacheServiceExceptionMapper.java
@@ -52,7 +52,6 @@ public static SdkException convert(Throwable e, Metadata metadata) {
 
       String errorCause = trailers.get(Metadata.Key.of("err", Metadata.ASCII_STRING_MARSHALLER));
       if (errorCause == null) {
-        // TODO remove once control service is updated to send "err" in metadata
         errorCause = grpcException.getMessage();
       }
 
@@ -82,9 +81,8 @@ public static SdkException convert(Throwable e, Metadata metadata) {
           return new LimitExceededException(grpcException, errorDetails);
 
         case NOT_FOUND:
-          if (errorCause.contains("element_not_found")) {
+          if (errorCause.contains("item_not_found")) {
             return new StoreItemNotFoundException(grpcException, errorDetails);
-            // TODO change once control service is updated to send "Store with name" in metadata
           } else if (errorCause.contains("Store with name")) {
             return new StoreNotFoundException(grpcException, errorDetails);
           } else {