Address PR comments

Signed-off-by: mgodwan <[email protected]>
opensearch-project · Feb 1, 2024 · 886757e · 886757e
1 parent 36925d2
commit 886757e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/server/src/main/java/org/opensearch/common/util/FeatureFlags.java b/server/src/main/java/org/opensearch/common/util/FeatureFlags.java
@@ -128,9 +128,5 @@ public static boolean isEnabled(Setting<Boolean> featureFlag) {
         Property.NodeScope
     );
 
-    public static final Setting<Boolean> DOC_ID_FUZZY_SET_SETTING = Setting.boolSetting(
-        DOC_ID_FUZZY_SET,
-        false,
-        Property.NodeScope
-    );
+    public static final Setting<Boolean> DOC_ID_FUZZY_SET_SETTING = Setting.boolSetting(DOC_ID_FUZZY_SET, false, Property.NodeScope);
 }
diff --git a/server/src/main/java/org/opensearch/index/IndexSettings.java b/server/src/main/java/org/opensearch/index/IndexSettings.java
@@ -1863,8 +1863,8 @@ private static <T> void verifyFeatureToSetDocIdFuzzySetSetting(Consumer<T> setti
         if (FeatureFlags.isEnabled(DOC_ID_FUZZY_SET_SETTING)) {
             settingUpdater.accept(val);
         } else {
-            throw new IllegalArgumentException(String.format("Fuzzy set for optimizing doc id lookup " +
-                "cannot be enabled with feature flag %s set to false", FeatureFlags.DOC_ID_FUZZY_SET));
+            throw new IllegalArgumentException("Fuzzy set for optimizing doc id lookup " +
+                "cannot be enabled with feature flag [" +  FeatureFlags.DOC_ID_FUZZY_SET + "] set to false");
         }
     }
 }