diff options
author | Jon Bratseth <bratseth@verizonmedia.com> | 2019-10-17 20:35:41 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@verizonmedia.com> | 2019-10-17 20:35:41 +0200 |
commit | 79c303d743daefb80e5a4d4e9f12b2c41a7aa69d (patch) | |
tree | 6ec08e682c0297f0b75c5869f4171527bbef8bfc | |
parent | b4689a871d31c1f47dd8ac8d6b4e1d16e1f8edd0 (diff) |
Check node just once per iteration
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java index e8d0839d60e..42b51f521bc 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/Rebalancer.java @@ -56,8 +56,8 @@ public class Rebalancer extends Maintainer { DockerHostCapacity capacity = new DockerHostCapacity(allNodes, hostResourcesCalculator); Move bestMove = Move.none; for (Node node : allNodes.state(Node.State.active)) { + if (node.parentHostname().isEmpty()) continue; for (Node toHost : allNodes.state(NodePrioritizer.ALLOCATABLE_HOST_STATES).nodeType(NodeType.host)) { - if (node.parentHostname().isEmpty()) continue; if (toHost.hostname().equals(node.parentHostname().get())) continue; if ( ! capacity.freeCapacityOf(toHost).satisfies(node.flavor().resources())) continue; |