From 0c43ea3655a74db93b303a0d5c3e366ded0bdf15 Mon Sep 17 00:00:00 2001 From: Daniel Widdis Date: Thu, 23 Mar 2023 10:29:36 -0700 Subject: [PATCH] Fix up comments and remove debug logging Signed-off-by: Daniel Widdis --- .../opensearch/extensions/action/ExtensionTransportAction.java | 2 +- .../extensions/action/ExtensionTransportActionsHandler.java | 3 --- 2 files changed, 1 insertion(+), 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 index 3d19b076812b3..4b0b9725e50ae 100644 --- 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 index 08f93cf5ce81a..3fba76b7d3c59 100644 --- 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) {