Allow all application hosts to talk to ZooKeeper

1 files changed, 10 insertions, 11 deletions

diff --git a/config-model/src/main/java/com/yahoo/config/model/provision/Host.java b/config-model/src/main/java/com/yahoo/config/model/provision/Host.java
index afe4de7ef0d..360853d0f79 100644
--- a/config-model/src/main/java/com/yahoo/config/model/provision/Host.java
+++ b/config-model/src/main/java/com/yahoo/config/model/provision/Host.java
@@ -1,40 +1,39 @@
 // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.config.model.provision;
 
+import com.google.common.collect.ImmutableList;
+
 import java.util.ArrayList;
 import java.util.List;
 
 /**
- * A hostname with zero or more aliases.
+ * A hostname with zero or more aliases. This is immutable.
  *
  * @author hmusum
  */
 public class Host {
 
     private final String hostname;
-    private final List<String> hostAliases;
+    private final ImmutableList<String> aliases;
 
     public Host(String hostname) {
         this.hostname = hostname;
-        this.hostAliases = new ArrayList<>();
+        this.aliases = ImmutableList.of();
     }
 
     public Host(String hostname, List<String> hostAliases) {
         this.hostname = hostname;
-        this.hostAliases = hostAliases;
+        this.aliases = ImmutableList.copyOf(hostAliases);
     }
 
-    public String getHostname() {
-        return hostname;
-    }
+    public String hostname() { return hostname; }
 
-    public List<String> getHostAliases() {
-        return hostAliases;
-    }
+    /** Returns an immutable list of the aliases of this node, which may be empty but never null */
+    public List<String> aliases() { return aliases; }
 
     @Override
     public String toString() {
-        return hostname + " (aliases: " + hostAliases + ")";
+        return hostname + (aliases.size() > 0 ? " (aliases: " + aliases + ")" : "" );
     }
 
 }