don't establish a connection between two client nodes
This commit is contained in:
parent
9d75849362
commit
9735d3e6d7
|
@ -223,6 +223,9 @@ public class InternalClusterService extends AbstractLifecycleComponent<ClusterSe
|
||||||
|
|
||||||
// TODO, do this in parallel (and wait)
|
// TODO, do this in parallel (and wait)
|
||||||
for (DiscoveryNode node : nodesDelta.addedNodes()) {
|
for (DiscoveryNode node : nodesDelta.addedNodes()) {
|
||||||
|
if (!nodeRequiresConnection(node)) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
try {
|
try {
|
||||||
transportService.connectToNode(node);
|
transportService.connectToNode(node);
|
||||||
} catch (Exception e) {
|
} catch (Exception e) {
|
||||||
|
@ -292,6 +295,9 @@ public class InternalClusterService extends AbstractLifecycleComponent<ClusterSe
|
||||||
if (lifecycle.stoppedOrClosed()) {
|
if (lifecycle.stoppedOrClosed()) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
if (!nodeRequiresConnection(node)) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
if (clusterState.nodes().nodeExists(node.id())) { // we double check existence of node since connectToNode might take time...
|
if (clusterState.nodes().nodeExists(node.id())) { // we double check existence of node since connectToNode might take time...
|
||||||
if (!transportService.nodeConnected(node)) {
|
if (!transportService.nodeConnected(node)) {
|
||||||
try {
|
try {
|
||||||
|
@ -309,4 +315,13 @@ public class InternalClusterService extends AbstractLifecycleComponent<ClusterSe
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private boolean nodeRequiresConnection(DiscoveryNode node) {
|
||||||
|
if (localNode().clientNode()) {
|
||||||
|
if (node.clientNode()) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return true;
|
||||||
|
}
|
||||||
}
|
}
|
Loading…
Reference in New Issue