Fix gradle failures

Signed-off-by: Ashish Singh <[email protected]>
opensearch-project · Mar 22, 2024 · 08448a5 · 08448a5
1 parent 49941fd
commit 08448a5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/server/src/main/java/org/opensearch/index/remote/RemoteStoreDataEnums.java b/server/src/main/java/org/opensearch/index/remote/RemoteStoreDataEnums.java
@@ -20,6 +20,9 @@
  */
 public class RemoteStoreDataEnums {
 
+    /**
+     * Categories of the data in Remote store.
+     */
     public enum DataCategory {
         SEGMENTS("segments", Set.of(DataType.values())),
         TRANSLOG("translog", Set.of(DATA, METADATA));
@@ -41,6 +44,9 @@ public String getName() {
         }
     }
 
+    /**
+     * Types of data in remote store.
+     */
     public enum DataType {
         DATA("data"),
         METADATA("metadata"),

diff --git a/server/src/test/java/org/opensearch/index/remote/RemoteStorePathTypeTests.java b/server/src/test/java/org/opensearch/index/remote/RemoteStorePathTypeTests.java
@@ -64,12 +64,12 @@ public void testGeneratePathForFixedType() {
         String basePath = getPath(pathList) + indexUUID + SEPARATOR + shardId + SEPARATOR;
         // Translog Data
         BlobPath result = FIXED.path(blobPath, indexUUID, shardId, dataCategory, dataType);
-        assertEquals(basePath + dataCategory + SEPARATOR + dataType + SEPARATOR, result.buildAsString());
+        assertEquals(basePath + dataCategory.getName() + SEPARATOR + dataType.getName() + SEPARATOR, result.buildAsString());
 
         // Translog Metadata
         dataType = METADATA;
         result = FIXED.path(blobPath, indexUUID, shardId, dataCategory, dataType);
-        assertEquals(basePath + dataCategory + SEPARATOR + dataType + SEPARATOR, result.buildAsString());
+        assertEquals(basePath + dataCategory.getName() + SEPARATOR + dataType.getName() + SEPARATOR, result.buildAsString());
 
         // Translog Lock files - This is a negative case where the assertion will trip.
         BlobPath finalBlobPath = blobPath;
@@ -79,17 +79,17 @@ public void testGeneratePathForFixedType() {
         dataCategory = SEGMENTS;
         dataType = DATA;
         result = FIXED.path(blobPath, indexUUID, shardId, dataCategory, dataType);
-        assertEquals(basePath + dataCategory + SEPARATOR + dataType + SEPARATOR, result.buildAsString());
+        assertEquals(basePath + dataCategory.getName() + SEPARATOR + dataType.getName() + SEPARATOR, result.buildAsString());
 
         // Segment Metadata
         dataType = METADATA;
         result = FIXED.path(blobPath, indexUUID, shardId, dataCategory, dataType);
-        assertEquals(basePath + dataCategory + SEPARATOR + dataType + SEPARATOR, result.buildAsString());
+        assertEquals(basePath + dataCategory.getName() + SEPARATOR + dataType.getName() + SEPARATOR, result.buildAsString());
 
         // Segment Metadata
         dataType = LOCK_FILES;
         result = FIXED.path(blobPath, indexUUID, shardId, dataCategory, dataType);
-        assertEquals(basePath + dataCategory + SEPARATOR + dataType + SEPARATOR, result.buildAsString());
+        assertEquals(basePath + dataCategory.getName() + SEPARATOR + dataType.getName() + SEPARATOR, result.buildAsString());
     }
 
     private List<String> getPathList() {