diff options
author | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-02-02 23:36:02 +0100 |
---|---|---|
committer | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-02-03 22:32:34 +0100 |
commit | 4fa56185803ebd2c7fad14a9a9ebc4ca6970d64f (patch) | |
tree | 6123299ff0d1f187b4158996fbb663d609e96f8c /node-repository/src | |
parent | 4d75505805c188366e3771446af4a99514433295 (diff) |
Fix potential NPE
Diffstat (limited to 'node-repository/src')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java index 85a43e38e07..dd90c52840b 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java @@ -218,7 +218,7 @@ public class NodePrioritizer { private boolean canStillAllocate(Node node) { if (node.type() != NodeType.tenant || node.parentHostname().isEmpty()) return true; Optional<Node> parent = allNodesAndHosts.parentOf(node); - return parent.isPresent() ? Nodes.canAllocateTenantNodeTo(parent.get(), dynamicProvisioning) : null; + return parent.isPresent() && Nodes.canAllocateTenantNodeTo(parent.get(), dynamicProvisioning); } } |