fix OR operator in DeleteModelTransportAction

Signed-off-by: Bhavana Ramaram <[email protected]>
rbhavna · Oct 4, 2023 · eb90be2 · eb90be2
1 parent 7d39173
commit eb90be2
Showing 1 changed file with 11 additions and 13 deletions.
diff --git a/plugin/src/main/java/org/opensearch/ml/action/models/DeleteModelTransportAction.java b/plugin/src/main/java/org/opensearch/ml/action/models/DeleteModelTransportAction.java
@@ -5,13 +5,10 @@
 
 package org.opensearch.ml.action.models;
 
-import static org.opensearch.core.xcontent.XContentParserUtils.ensureExpectedToken;
-import static org.opensearch.ml.common.CommonValue.ML_MODEL_INDEX;
-import static org.opensearch.ml.common.MLModel.ALGORITHM_FIELD;
-import static org.opensearch.ml.common.MLModel.MODEL_ID_FIELD;
-import static org.opensearch.ml.utils.MLNodeUtils.createXContentParserFromRegistry;
-import static org.opensearch.ml.utils.RestActionUtils.getFetchSourceContext;
-
+import com.google.common.annotations.VisibleForTesting;
+import lombok.AccessLevel;
+import lombok.experimental.FieldDefaults;
+import lombok.extern.log4j.Log4j2;
 import org.opensearch.OpenSearchStatusException;
 import org.opensearch.ResourceNotFoundException;
 import org.opensearch.action.ActionRequest;
@@ -46,11 +43,12 @@
 import org.opensearch.tasks.Task;
 import org.opensearch.transport.TransportService;
 
-import com.google.common.annotations.VisibleForTesting;
-
-import lombok.AccessLevel;
-import lombok.experimental.FieldDefaults;
-import lombok.extern.log4j.Log4j2;
+import static org.opensearch.core.xcontent.XContentParserUtils.ensureExpectedToken;
+import static org.opensearch.ml.common.CommonValue.ML_MODEL_INDEX;
+import static org.opensearch.ml.common.MLModel.ALGORITHM_FIELD;
+import static org.opensearch.ml.common.MLModel.MODEL_ID_FIELD;
+import static org.opensearch.ml.utils.MLNodeUtils.createXContentParserFromRegistry;
+import static org.opensearch.ml.utils.RestActionUtils.getFetchSourceContext;
 
 @Log4j2
 @FieldDefaults(level = AccessLevel.PRIVATE)
@@ -115,7 +113,7 @@ protected void doExecute(Task task, ActionRequest request, ActionListener<Delete
                                     || mlModelState.equals(MLModelState.LOADING)
                                     || mlModelState.equals(MLModelState.PARTIALLY_LOADED)
                                     || mlModelState.equals(MLModelState.DEPLOYED)
-                                    || mlModelState.equals(MLModelState.DEPLOYING) | mlModelState.equals(MLModelState.PARTIALLY_DEPLOYED)) {
+                                    || mlModelState.equals(MLModelState.DEPLOYING) || mlModelState.equals(MLModelState.PARTIALLY_DEPLOYED)) {
                                     actionListener
                                         .onFailure(
                                             new Exception(