Refactor to prepare for mving more code to zokeeper-server-common

3 files changed, 12 insertions, 1 deletions

diff --git a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/DummyVespaZooKeeperServer.java b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/DummyVespaZooKeeperServer.java
index 15e888df97e..88968cc0de9 100644
--- a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/DummyVespaZooKeeperServer.java
+++ b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/DummyVespaZooKeeperServer.java
@@ -4,6 +4,8 @@ package com.yahoo.vespa.zookeeper;
 import com.google.inject.Inject;
 import com.yahoo.component.AbstractComponent;
 
+import java.nio.file.Path;
+
 /**
  * A dummy {@link VespaZooKeeperServer} implementation that can be used when a container cluster is not configured with standalone ZK cluster.
  *
@@ -13,4 +15,7 @@ public class DummyVespaZooKeeperServer extends AbstractComponent implements Vesp
 
     @Inject public DummyVespaZooKeeperServer() {}
 
+    @Override
+    public void start(Path path) {}
+
 }
diff --git a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/ReconfigException.java b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/ReconfigException.java
index ade9245615b..c4bb09eeedf 100644
--- a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/ReconfigException.java
+++ b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/ReconfigException.java
@@ -2,7 +2,8 @@
 package com.yahoo.vespa.zookeeper;
 
 /**
- * Interface for reconfiguring a zookeeper cluster.
+ * Exception used to wrap zookeeper exception when reconfiguration fails in a
+ * class that can be used without depending on ZooKeeper.
  *
  * @author hmusum
  */
@@ -16,4 +17,5 @@ public class ReconfigException extends RuntimeException {
     public ReconfigException(String message) {
         super(message);
     }
+
 }
diff --git a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/VespaZooKeeperServer.java b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/VespaZooKeeperServer.java
index 5a1f31a75c3..dbe4d4987a4 100644
--- a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/VespaZooKeeperServer.java
+++ b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/VespaZooKeeperServer.java
@@ -1,6 +1,8 @@
 // Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.zookeeper;
 
+import java.nio.file.Path;
+
 /**
  * Interface for a component that writes zookeeper config and starts a zookeeper server.
  *
@@ -8,4 +10,6 @@ package com.yahoo.vespa.zookeeper;
  */
 public interface VespaZooKeeperServer {
 
+    void start(Path path);
+
 }