do not return model and task id in response (#171)

Signed-off-by: Xun Zhang <[email protected]>

Author: Zhangxunmt

plugin/src/main/java/org/opensearch/ml/action/models/GetModelTransportAction.java

@@ -71,11 +71,11 @@ protected void doExecute(Task task, ActionRequest request, ActionListener<MLMode
                         actionListener.onFailure(e);
                     }
                 } else {
-                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find model " + modelId));
+                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find model"));
                 }
             }, e -> {
                 if (e instanceof IndexNotFoundException) {
-                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find model " + modelId));
+                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find model"));
                 } else {
                     log.error("Failed to get ML model " + modelId, e);
                     actionListener.onFailure(e);

plugin/src/main/java/org/opensearch/ml/action/tasks/GetTaskTransportAction.java

@@ -68,11 +68,11 @@ protected void doExecute(Task task, ActionRequest request, ActionListener<MLTask
                         actionListener.onFailure(e);
                     }
                 } else {
-                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find task " + taskId));
+                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find task"));
                 }
             }, e -> {
                 if (e instanceof IndexNotFoundException) {
-                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find task " + taskId));
+                    actionListener.onFailure(new MLResourceNotFoundException("Fail to find task"));
                 } else {
                     log.error("Failed to get ML task " + taskId, e);
                     actionListener.onFailure(e);

plugin/src/main/java/org/opensearch/ml/action/models/SearchTaskTransportAction.java renamed to plugin/src/main/java/org/opensearch/ml/action/tasks/SearchTaskTransportAction.java

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-package org.opensearch.ml.action.models;
+package org.opensearch.ml.action.tasks;
 
 import org.opensearch.action.ActionListener;
 import org.opensearch.action.search.SearchRequest;

plugin/src/main/java/org/opensearch/ml/plugin/MachineLearningPlugin.java

@@ -33,12 +33,12 @@
 import org.opensearch.ml.action.models.DeleteModelTransportAction;
 import org.opensearch.ml.action.models.GetModelTransportAction;
 import org.opensearch.ml.action.models.SearchModelTransportAction;
-import org.opensearch.ml.action.models.SearchTaskTransportAction;
 import org.opensearch.ml.action.prediction.TransportPredictionTaskAction;
 import org.opensearch.ml.action.stats.MLStatsNodesAction;
 import org.opensearch.ml.action.stats.MLStatsNodesTransportAction;
 import org.opensearch.ml.action.tasks.DeleteTaskTransportAction;
 import org.opensearch.ml.action.tasks.GetTaskTransportAction;
+import org.opensearch.ml.action.tasks.SearchTaskTransportAction;
 import org.opensearch.ml.action.training.TransportTrainingTaskAction;
 import org.opensearch.ml.action.trainpredict.TransportTrainAndPredictionTaskAction;
 import org.opensearch.ml.common.breaker.MLCircuitBreakerService;

plugin/src/test/java/org/opensearch/ml/rest/RestMLGetModelActionIT.java

@@ -22,7 +22,7 @@ public class RestMLGetModelActionIT extends MLCommonsRestTestCase {
 
     public void testGetModelAPI_EmptyResources() throws IOException {
         exceptionRule.expect(ResponseException.class);
-        exceptionRule.expectMessage("Fail to find model 111222333");
+        exceptionRule.expectMessage("Fail to find model");
         TestHelper.makeRequest(client(), "GET", "/_plugins/_ml/models/111222333", null, "", null);
     }