Added javadoc based on feedback

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

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

@@ -20,6 +20,7 @@ public class DeleteDataObjectRequest {
      * For data storage implementations other than OpenSearch, an index may be referred to as a table and the id may be referred to as a primary key.
      * @param index the index location to delete the object
      * @param id the document id
+     * @param tenantId the tenant id
      */
     public DeleteDataObjectRequest(String index, String id, String tenantId) {
         this.index = index;

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

@@ -23,6 +23,7 @@ public class GetDataObjectRequest {
      * For data storage implementations other than OpenSearch, an index may be referred to as a table and the id may be referred to as a primary key.
      * @param index the index location to get the object
      * @param id the document id
+     * @param tenantId the tenant id
      * @param fetchSourceContext the context to use when fetching _source
      */
     public GetDataObjectRequest(String index, String id, String tenantId, FetchSourceContext fetchSourceContext) {

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

@@ -23,6 +23,7 @@ public class UpdateDataObjectRequest {
      * For data storage implementations other than OpenSearch, an index may be referred to as a table and the data object may be referred to as an item.
      * @param index the index location to update the object
      * @param id the document id
+     * @param tenantId the tenant id
      * @param dataObject the data object
      */
     public UpdateDataObjectRequest(String index, String id, String tenantId, ToXContentObject dataObject) {
@@ -100,7 +101,7 @@ public Builder id(String id) {
 
          /**
          * Add a tenant ID to this builder
-         * @param id the tenant id
+         * @param tenantId the tenant id
          * @return the updated builder
          */
         public Builder tenantId(String tenantId) {

plugin/src/main/java/org/opensearch/ml/sdkclient/SdkClientModule.java

@@ -54,6 +54,7 @@ public SdkClientModule() {
 
     /**
      * Instantiate this module specifying the endpoint and region. Package private for testing.
+     * @param remoteMetadataType Type of remote metadata store
      * @param remoteMetadataEndpoint The remote endpoint
      * @param region The region
      */

plugin/src/test/java/org/opensearch/ml/sdkclient/DDBOpenSearchClientTests.java

@@ -27,7 +27,6 @@
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.opensearch.OpenSearchException;
 import org.opensearch.common.settings.Settings;
 import org.opensearch.common.unit.TimeValue;
 import org.opensearch.common.util.concurrent.OpenSearchExecutors;
@@ -165,7 +164,7 @@ public void testPutDataObject_DDBException_ThrowsException() {
             .toCompletableFuture();
 
         CompletionException ce = assertThrows(CompletionException.class, () -> future.join());
-        assertEquals(OpenSearchException.class, ce.getCause().getClass());
+        assertEquals(RuntimeException.class, ce.getCause().getClass());
     }
 
     @Test
@@ -227,7 +226,7 @@ public void testGetDataObject_DDBException_ThrowsOSException() throws IOExceptio
             .getDataObjectAsync(getRequest, testThreadPool.executor(GENERAL_THREAD_POOL))
             .toCompletableFuture();
         CompletionException ce = assertThrows(CompletionException.class, () -> future.join());
-        assertEquals(OpenSearchException.class, ce.getCause().getClass());
+        assertEquals(RuntimeException.class, ce.getCause().getClass());
     }
 
     @Test