summaryrefslogtreecommitdiffstats
path: root/node-repository
diff options
context:
space:
mode:
authortoby <smorgrav@yahoo-inc.com>2017-09-07 09:42:06 -0700
committertoby <smorgrav@yahoo-inc.com>2017-09-07 09:42:06 -0700
commit59d9761b9b5beb60af4554ff5d2790957c545a10 (patch)
tree50bc849ed80cbeed035db35c6b17f9f90ff31c05 /node-repository
parent22b5bd7c1d638b09a707cbe31762b3f33de8ff18 (diff)
Remove dead code and tune javadoc
Diffstat (limited to 'node-repository')
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/NameResolver.java4
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizer.java5
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()