use static max heap threshold 100

Signed-off-by: Xun Zhang <xunzh@amazon.com>

Author: Zhangxunmt

plugin/src/main/java/org/opensearch/ml/breaker/MemoryCircuitBreaker.java

@@ -18,6 +18,7 @@ public class MemoryCircuitBreaker extends ThresholdCircuitBreaker<Short> {
     // TODO: make this value configurable as cluster setting
     private static final String ML_MEMORY_CB = "Memory Circuit Breaker";
     public static final short DEFAULT_JVM_HEAP_USAGE_THRESHOLD = 85;
+    public static final short JVM_HEAP_MAX_THRESHOLD = 100; // when threshold is 100, this CB check is ignored
     private final JvmService jvmService;
     private volatile Integer jvmHeapMemThreshold = 85;
 
@@ -45,6 +46,6 @@ public String getName() {
 
     @Override
     public boolean isOpen() {
-        return getThreshold() < 100 && jvmService.stats().getMem().getHeapUsedPercent() > getThreshold();
+        return getThreshold() < JVM_HEAP_MAX_THRESHOLD && jvmService.stats().getMem().getHeapUsedPercent() > getThreshold();
     }
 }

plugin/src/test/java/org/opensearch/ml/model/MLModelManagerTests.java

@@ -1018,13 +1018,13 @@ private MLRegisterModelInput mockPretrainedInput() {
 
     private MLRegisterModelInput mockRemoteModelInput(boolean isHidden) {
         return MLRegisterModelInput
-                .builder()
-                .modelName(modelName)
-                .version(version)
-                .modelGroupId("modelGroupId")
-                .modelFormat(modelFormat)
-                .functionName(FunctionName.REMOTE)
-                .deployModel(true)
-                .build();
+            .builder()
+            .modelName(modelName)
+            .version(version)
+            .modelGroupId("modelGroupId")
+            .modelFormat(modelFormat)
+            .functionName(FunctionName.REMOTE)
+            .deployModel(true)
+            .build();
     }
 }