diff options
author | toby <smorgrav@yahoo-inc.com> | 2017-09-07 09:42:06 -0700 |
---|---|---|
committer | toby <smorgrav@yahoo-inc.com> | 2017-09-07 09:42:06 -0700 |
commit | 59d9761b9b5beb60af4554ff5d2790957c545a10 (patch) | |
tree | 50bc849ed80cbeed035db35c6b17f9f90ff31c05 /node-repository | |
parent | 22b5bd7c1d638b09a707cbe31762b3f33de8ff18 (diff) |
Remove dead code and tune javadoc
Diffstat (limited to 'node-repository')
2 files changed, 3 insertions, 6 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/NameResolver.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/NameResolver.java index 1b6e4628927..32451bf4946 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/NameResolver.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/NameResolver.java @@ -5,7 +5,7 @@ import java.util.Optional; import java.util.Set; /** - * Interface for a basic name to IP address resolver. + * Interface for a hostname/ip resolver. * * @author mpolden */ @@ -20,7 +20,7 @@ public interface NameResolver { Set<String> getAllByNameOrThrow(String hostname); /** - * Get hostname from IP address + * Get hostname from IP address. * * @param ipAddress The IPv4 or IPv6 address for the host * @return Empty if the IP does not resolve or the hostname if it does 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 b30f8a8c020..7d5cd85a783 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 @@ -43,9 +43,6 @@ public class NodePrioritizer { private final Set<Node> spareHosts; private final Map<Node, ResourceCapacity> headroomHosts; - /** This is before we mock DNS to resolve on test IP that we use in tests */ - public static boolean unitTesting = false; - NodePrioritizer(List<Node> allNodes, ApplicationId appId, ClusterSpec clusterSpec, NodeSpec nodeSpec, NodeFlavors nodeFlavors, int spares, NameResolver nameResolver) { this.allNodes = Collections.unmodifiableList(allNodes); this.requestedNodes = nodeSpec; @@ -221,7 +218,7 @@ public class NodePrioritizer { } /** - * Add nodes already provisioned, but not allocatied to any application + * Add nodes already provisioned, but not allocated to any application */ void addReadyNodes() { allNodes.stream() |