nit fixes

Signed-off-by: Sarthak Aggarwal <sarthagg@amazon.com>
sarthakaggarwal97 · Sep 6, 2023 · 35e7083 · 35e7083
1 parent ca32bf4
commit 35e7083
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 12 deletions.
diff --git a/server/src/internalClusterTest/java/org/opensearch/index/codec/ZstdNotEnabledIT.java b/server/src/internalClusterTest/java/org/opensearch/index/codec/ZstdNotEnabledIT.java
@@ -12,6 +12,8 @@
 import org.opensearch.common.settings.Settings;
 import org.opensearch.test.OpenSearchIntegTestCase;
 
+import java.util.List;
+
 @OpenSearchIntegTestCase.ClusterScope(scope = OpenSearchIntegTestCase.Scope.TEST)
 public class ZstdNotEnabledIT extends OpenSearchIntegTestCase {
 
@@ -20,18 +22,20 @@ public void testZStdCodecsWithoutPluginInstalled() {
         internalCluster().startNode();
         final String index = "test-index";
 
-        // creating index with zstd or zstd_no_dict should fail if custom-codecs plugin is not installed
-        assertThrows(
-            IllegalArgumentException.class,
-            () -> createIndex(
-                index,
-                Settings.builder()
-                    .put(IndexMetadata.SETTING_NUMBER_OF_SHARDS, 1)
-                    .put(IndexMetadata.SETTING_NUMBER_OF_REPLICAS, 0)
-                    .put("index.codec", randomFrom("zstd", "zstd_no_dict"))
-                    .build()
-            )
-        );
+        // creating index with zstd and zstd_no_dict should fail if custom-codecs plugin is not installed
+        for (String codec : List.of("zstd", "zstd_no_dict")) {
+            assertThrows(
+                IllegalArgumentException.class,
+                () -> createIndex(
+                    index,
+                    Settings.builder()
+                        .put(IndexMetadata.SETTING_NUMBER_OF_SHARDS, 1)
+                        .put(IndexMetadata.SETTING_NUMBER_OF_REPLICAS, 0)
+                        .put("index.codec", codec)
+                        .build()
+                )
+            );
+        }
     }
 
 }
diff --git a/server/src/main/java/org/opensearch/index/codec/CodecAliases.java b/server/src/main/java/org/opensearch/index/codec/CodecAliases.java
@@ -9,6 +9,7 @@
 package org.opensearch.index.codec;
 
 import org.apache.lucene.codecs.Codec;
+import org.opensearch.common.annotation.ExperimentalApi;
 
 import java.util.Set;
 
@@ -17,6 +18,7 @@
  *
  * @opensearch.internal
  */
+@ExperimentalApi
 public interface CodecAliases {
 
     /**