Minor cleanup

We don't use CVS anymore
Rename a couple of methods

2 files changed, 7 insertions, 9 deletions

diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperClient.java b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperClient.java
index 42f4dbd5762..f0a63757477 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperClient.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperClient.java
@@ -35,7 +35,7 @@ import static com.yahoo.vespa.config.server.zookeeper.ConfigCurator.USERAPP_ZK_S
 import static com.yahoo.vespa.config.server.zookeeper.ConfigCurator.USER_DEFCONFIGS_ZK_SUBPATH;
 
 /**
- * A class used for reading and writing application data to zookeeper.
+ * Reads and writes application package to and from ZooKeeper.
  *
  * @author hmusum
  */
@@ -76,13 +76,13 @@ public class ZooKeeperClient {
      *
      * @param app the application package to feed to zookeeper
      */
-    void write(ApplicationPackage app) {
+    void writeApplicationPackage(ApplicationPackage app) {
         try {
             writeUserDefs(app);
             writeSomeOf(app);
             writeSchemas(app);
             writeUserIncludeDirs(app, app.getUserIncludeDirs());
-            write(app.getMetaData());
+            writeMetadata(app.getMetaData());
         } catch (Exception e) {
             throw new IllegalStateException("Unable to write vespa model to config server(s) " + System.getProperty("configsources") + "\n" +
                                             "Please ensure that config server is started " +
@@ -153,7 +153,6 @@ public class ZooKeeperClient {
         for (ApplicationFile file : listFiles(dir, filenameFilter)) {
             String name = file.getPath().getName();
             if (name.startsWith(".")) continue; //.svn , .git ...
-            if ("CVS".equals(name)) continue;
             if (file.isDirectory()) {
                 configCurator.createNode(path.append(name).getAbsolute());
                 if (recurse) {
@@ -198,7 +197,6 @@ public class ZooKeeperClient {
     }
 
     private void writeUserIncludeDirs(ApplicationPackage applicationPackage, List<String> userIncludeDirs) throws IOException {
-        // User defined include directories
         for (String userInclude : userIncludeDirs) {
             ApplicationFile dir = applicationPackage.getFile(Path.fromString(userInclude));
             final List<ApplicationFile> files = dir.listFiles();
@@ -238,12 +236,12 @@ public class ZooKeeperClient {
     }
 
     /**
-     * Feeds application metadata to zookeeper. Used by vespamodel to create config
-     * for application metadata (used by ApplicationStatusHandler)
+     * Feeds application metadata to zookeeper. Used by config model to create config
+     * for application metadata
      *
      * @param metaData The application metadata.
      */
-    private void write(ApplicationMetaData metaData) {
+    private void writeMetadata(ApplicationMetaData metaData) {
         configCurator.putData(getZooKeeperAppPath(META_ZK_PATH).getAbsolute(), metaData.asJsonBytes());
     }
 
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperDeployer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperDeployer.java
index 12aa5b7cc35..8c7d6ea28dd 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperDeployer.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/ZooKeeperDeployer.java
@@ -33,7 +33,7 @@ public class ZooKeeperDeployer {
     public void deploy(ApplicationPackage applicationPackage, Map<Version, FileRegistry> fileRegistryMap, 
                        AllocatedHosts allocatedHosts) throws IOException {
         zooKeeperClient.initialize();
-        zooKeeperClient.write(applicationPackage);
+        zooKeeperClient.writeApplicationPackage(applicationPackage);
         zooKeeperClient.write(fileRegistryMap);
         zooKeeperClient.write(allocatedHosts);
     }