Fix up comments and remove debug logging

Signed-off-by: Daniel Widdis <widdis@gmail.com>
opensearch-project · Mar 23, 2023 · 0c43ea3 · 0c43ea3
1 parent 8b2f179
commit 0c43ea3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/server/src/main/java/org/opensearch/extensions/action/ExtensionTransportAction.java b/server/src/main/java/org/opensearch/extensions/action/ExtensionTransportAction.java
@@ -16,7 +16,7 @@
 import org.opensearch.tasks.TaskManager;
 
 /**
- * A proxy transport action used to proxy a transport request from OpenSearch or a plugin to execute on an extension
+ * A proxy transport action used to proxy a transport request from an extension to execute on another extension
  *
  * @opensearch.internal
  */

diff --git a/server/src/main/java/org/opensearch/extensions/action/ExtensionTransportActionsHandler.java b/server/src/main/java/org/opensearch/extensions/action/ExtensionTransportActionsHandler.java
@@ -96,7 +96,6 @@ public DiscoveryExtensionNode getExtension(String action) {
         if (uniqueId == null) {
             throw new ActionNotFoundTransportException(action);
         }
-        logger.info("Got extension: " + extensionIdMap.get(uniqueId));
         return extensionIdMap.get(uniqueId);
     }
 
@@ -107,10 +106,8 @@ public DiscoveryExtensionNode getExtension(String action) {
      * @return  A {@link AcknowledgedResponse} indicating success.
      */
     public TransportResponse handleRegisterTransportActionsRequest(RegisterTransportActionsRequest transportActionsRequest) {
-        logger.debug("Register Transport Actions request recieved {}", transportActionsRequest);
         try {
             for (String action : transportActionsRequest.getTransportActions()) {
-                logger.info("Registering action " + action + " with id " + transportActionsRequest.getUniqueId());
                 registerAction(action, transportActionsRequest.getUniqueId());
             }
         } catch (Exception e) {