Address PR comments

Signed-off-by: Sachin Kale <[email protected]>
sachinpkale · Sep 2, 2024 · 38c36b9 · 38c36b9
1 parent 227686f
commit 38c36b9
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/server/src/main/java/org/opensearch/node/remotestore/RemoteStorePinnedTimestampService.java b/server/src/main/java/org/opensearch/node/remotestore/RemoteStorePinnedTimestampService.java
@@ -121,6 +121,7 @@ public void pinTimestamp(long timestamp, String pinningEntity, ActionListener<Vo
                     elapsedTime,
                     RemoteStoreSettings.getPinnedTimestampsLookbackInterval().nanos()
                 );
+                logger.error(errorMessage);
                 unpinTimestamp(timestamp, pinningEntity, ActionListener.wrap(() -> listener.onFailure(new RuntimeException(errorMessage))));
             } else {
                 listener.onResponse(null);
@@ -165,6 +166,7 @@ public void cloneTimestamp(long timestamp, String existingPinningEntity, String
                     timestamp,
                     existingPinningEntity
                 );
+                logger.error(errorMessage);
                 listener.onFailure(new IllegalArgumentException(errorMessage));
             }
         } catch (IOException e) {
@@ -205,6 +207,7 @@ public void unpinTimestamp(long timestamp, String pinningEntity, ActionListener<
                 listener.onResponse(null);
             } else {
                 String errorMessage = String.format(Locale.ROOT, "Timestamp: %s is not pinned by entity: %s", timestamp, pinningEntity);
+                logger.error(errorMessage);
                 listener.onFailure(new IllegalArgumentException(errorMessage));
             }
         } catch (IOException e) {