diff options
author | gjoranv <gv@verizonmedia.com> | 2019-03-21 14:26:23 +0100 |
---|---|---|
committer | gjoranv <gv@verizonmedia.com> | 2019-03-21 14:26:23 +0100 |
commit | a2a9e43ac40bcb4f16f82d57d29d80a6cf9fa830 (patch) | |
tree | 223329d9342750a519e383cf0d9c0d9142417ceb /config-model | |
parent | 2157eac25b44ffee43e2c88488c71dc23a576abe (diff) |
Use 'index' from super class.
Diffstat (limited to 'config-model')
3 files changed, 4 insertions, 9 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerCluster.java index 88196e5ac4f..61a42d1c1a6 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerCluster.java @@ -37,7 +37,7 @@ public class ClusterControllerCluster extends AbstractConfigProducer<ClusterCont for (ClusterControllerContainer container : containerCluster.getContainers()) { ZookeeperServerConfig.Server.Builder serverBuilder = new ZookeeperServerConfig.Server.Builder(); serverBuilder.hostname(container.getHostName()); - serverBuilder.id(container.getIndex()); + serverBuilder.id(container.index()); builder.server(serverBuilder); } } diff --git a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java index 7c08f386e94..46acdb44746 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java @@ -32,13 +32,11 @@ public class ClusterControllerContainer extends Container implements { private static final ComponentSpecification CLUSTERCONTROLLER_BUNDLE = new ComponentSpecification("clustercontroller-apps"); private static final ComponentSpecification ZKFACADE_BUNDLE = new ComponentSpecification("zkfacade"); - private final int index; private final Set<String> bundles = new TreeSet<>(); public ClusterControllerContainer(AbstractConfigProducer parent, int index, boolean runStandaloneZooKeeper, boolean isHosted) { super(parent, "" + index, index); - this.index = index; addHandler( new Handler(new ComponentModel(new BundleInstantiationSpecification( new ComponentSpecification("clustercontroller-status"), @@ -80,7 +78,7 @@ public class ClusterControllerContainer extends Container implements @Override public boolean requiresWantedPort() { - return index == 0; + return index() == 0; } @Override @@ -107,7 +105,7 @@ public class ClusterControllerContainer extends Container implements @Override public void getConfig(ZookeeperServerConfig.Builder builder) { - builder.myid(index); + builder.myid(index()); } @Override @@ -115,7 +113,4 @@ public class ClusterControllerContainer extends Container implements builder.jvm(new QrStartConfig.Jvm.Builder().heapsize(512)); } - int getIndex() { - return index; - } } diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/Container.java b/config-model/src/main/java/com/yahoo/vespa/model/container/Container.java index 8e0a06ad69c..a1104510462 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/Container.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/Container.java @@ -134,7 +134,7 @@ public abstract class Container extends AbstractService implements } /** Returns the index of this node. The index of a given node is stable through changes with best effort. */ - public int index() { return index; } + public final int index() { return index; } // We cannot set bindings yet, as baseport is not initialized public void addBuiltinHandlers() { } |