diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-12-09 13:47:55 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-12-09 13:47:55 +0100 |
commit | 34a4923ad48f25610a5f7e00a57b840f4f7f1197 (patch) | |
tree | 403c5a91bcdb353eadc8e124888e2e5cefd65cdf /zookeeper-server | |
parent | 70435c1b3b7f58eba996da4e489878c4430836c0 (diff) |
Revert "Set reconfigEnabled based on config value"
Diffstat (limited to 'zookeeper-server')
2 files changed, 1 insertions, 3 deletions
diff --git a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Configurator.java b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Configurator.java index 0e3f3743c18..df4b4e76c0b 100644 --- a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Configurator.java +++ b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Configurator.java @@ -90,7 +90,7 @@ public class Configurator { sb.append("serverCnxnFactory=org.apache.zookeeper.server.NettyServerCnxnFactory").append("\n"); sb.append("quorumListenOnAllIPs=true").append("\n"); sb.append("standaloneEnabled=false").append("\n"); - sb.append("reconfigEnabled=" + config.dynamicReconfiguration()).append("\n"); + sb.append("reconfigEnabled=true").append("\n"); sb.append("skipACL=yes").append("\n"); ensureThisServerIsRepresented(config.myid(), config.server()); config.server().forEach(server -> addServerToCfg(sb, server, config.clientPort())); diff --git a/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ConfiguratorTest.java b/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ConfiguratorTest.java index 04a4010edf8..a106b6ba99e 100644 --- a/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ConfiguratorTest.java +++ b/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ConfiguratorTest.java @@ -69,7 +69,6 @@ public class ConfiguratorTest { builder.server(newServer(2, "baz", 345, 543)); builder.myidFile(idFile.getAbsolutePath()); builder.myid(1); - builder.dynamicReconfiguration(true); new Configurator(builder.build()).writeConfigToDisk(Optional.empty()); validateConfigFileMultipleHosts(cfgFile); validateIdFile(idFile, "1\n"); @@ -115,7 +114,6 @@ public class ConfiguratorTest { builder.server(newServer(0, "foo", 123, 321)); builder.myid(0); builder.jksKeyStoreFile(jksKeyStoreFile.getAbsolutePath()); - builder.dynamicReconfiguration(true); return builder; } |