Add return to avoid unnecessary computation (opensearch-project#6615)

Signed-off-by: Bukhtawar Khan <[email protected]>
bharath-techie · Mar 12, 2023 · c2388b5 · c2388b5
1 parent 671f1ba
commit c2388b5
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/...rc/main/java/org/opensearch/action/admin/cluster/health/TransportClusterHealthAction.java b/...rc/main/java/org/opensearch/action/admin/cluster/health/TransportClusterHealthAction.java
@@ -283,6 +283,7 @@ private void executeHealth(
             if (request.ensureNodeWeighedIn()) {
                 if (clusterHealthResponse.hasDiscoveredClusterManager() == false) {
                     listener.onFailure(new ClusterManagerNotDiscoveredException("cluster-manager not discovered"));
+                    return;
                 } else {
                     DiscoveryNode localNode = currentState.getNodes().getLocalNode();
                     // TODO: make this check more generic, check for node role instead