Addressed comments

Signed-off-by: Arjun kumar Giri <[email protected]>

common/src/main/java/org/opensearch/sdk/UpdateDataObjectRequest.java

@@ -136,7 +136,7 @@ public Builder id(String id) {
             return this;
         }
 
-         /**
+        /**
          * Add a tenant ID to this builder
          * @param tenantId the tenant id
          * @return the updated builder

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

@@ -437,6 +437,8 @@ public MachineLearningPlugin(Settings settings) {
 
     @Override
     public Collection<Module> createGuiceModules() {
+        // TODO: SDKClientModule is initialized both in createGuiceModules and createComponents. Unify these
+        // approaches to prevent multiple instances of SDKClient.
         return List.of(new SdkClientModule(null, null));
     }
 
@@ -464,6 +466,7 @@ public Collection<Object> createComponents(
         Settings settings = environment.settings();
         Path dataPath = environment.dataFiles()[0];
         Path configFile = environment.configFile();
+        // TODO: Rather than recreating SDKClientModule reuse module created as part of createGuiceModules
         ModulesBuilder modules = new ModulesBuilder();
         modules.add(new SdkClientModule(client, xContentRegistry));
         Injector injector = modules.createInjector();

plugin/src/test/java/org/opensearch/ml/action/controller/GetControllerTransportActionTests.java

@@ -5,7 +5,9 @@
 
 package org.opensearch.ml.action.controller;
 
-import static org.mockito.ArgumentMatchers.*;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.eq;
+import static org.mockito.ArgumentMatchers.isA;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.verify;