Nonfunctional changes only

7 files changed, 22 insertions, 78 deletions

diff --git a/config-model/src/main/java/com/yahoo/vespa/model/Client.java b/config-model/src/main/java/com/yahoo/vespa/model/Client.java
index 15685f5f669..2a2498cc310 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/Client.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/Client.java
@@ -8,7 +8,7 @@ import com.yahoo.config.model.producer.AbstractConfigProducer;
  * This is a placeholder config producer that makes global configuration available through a single identifier. This
  * is added directly to the {@link ApplicationConfigProducerRoot} producer, and so can be accessed by the simple "client" identifier.
  *
- * @author <a href="mailto:simon@yahoo-inc.com">Simon Thoresen</a>
+ * @author Simon Thoresen
  */
 public class Client extends AbstractConfigProducer {
 
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/ConfigProducer.java b/config-model/src/main/java/com/yahoo/vespa/model/ConfigProducer.java
index 852e4e73331..aaeedf10bc8 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/ConfigProducer.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/ConfigProducer.java
@@ -19,44 +19,38 @@ import com.yahoo.config.model.producer.UserConfigRepo;
  */
 public interface ConfigProducer extends com.yahoo.config.ConfigInstance.Producer {
 
-    /**
-     * @return the configId of this ConfigProducer.
-     */
-    public String getConfigId();
+    /** Returns the configId of this ConfigProducer. */
+    String getConfigId();
 
-    /**
-     * @return  The one and only HostSystem of the root node
-     */
-    public HostSystem getHostSystem();
+    /** Returns the one and only HostSystem of the root node */
+    HostSystem getHostSystem();
 
     /** Returns the user configs of this */
-    public UserConfigRepo getUserConfigs();
+    UserConfigRepo getUserConfigs();
     
-    /**
-     * @return this ConfigProducer's children (only 1st level)
-     */
-    public Map<String,? extends ConfigProducer> getChildren();
+    /** Returns this ConfigProducer's children (only 1st level) */
+    Map<String,? extends ConfigProducer> getChildren();
 
-    /**
-     * @return a List of all Services that are descendants to this ConfigProducer
-     */
-    public List<Service> getDescendantServices();
+    /** Returns a List of all Services that are descendants to this ConfigProducer */
+    List<Service> getDescendantServices();
 
     /**
      * Writes files that need to be written.  The files will usually
      * only be written when the Vespa model is generated through the
      * deploy-application script.
-     * gv: This is primarily intended for debugging.
+     * This is primarily intended for debugging.
+     * 
      * @param directory directory to write files to
      * @throws java.io.IOException if writing fails
      */
-    public void writeFiles(File directory) throws IOException;
+    void writeFiles(File directory) throws IOException;
 
     /**
      * Dump the three of config producers to the specified stream.
+     * 
      * @param out The stream to print to, e.g. System.out
      */
-    public void dump(PrintStream out);
+    void dump(PrintStream out);
 
     /**
      * Build config from this and all parent ConfigProducers,
@@ -74,11 +68,12 @@ public interface ConfigProducer extends com.yahoo.config.ConfigInstance.Producer
      * @param builder The ConfigBuilder to add user config overrides.
      * @return true if overrides were added, false if not.
      */
-    public boolean addUserConfig(ConfigInstance.Builder builder);
+    boolean addUserConfig(ConfigInstance.Builder builder);
 
     /**
      * check constraints depending on the state of the vespamodel graph.
      * When overriding, you must invoke super.
      */
-    public void validate() throws Exception;
+    void validate() throws Exception;
+
 }
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/PlainFormatter.java b/config-model/src/main/java/com/yahoo/vespa/model/PlainFormatter.java
deleted file mode 100644
index d424f4fa31b..00000000000
--- a/config-model/src/main/java/com/yahoo/vespa/model/PlainFormatter.java
+++ /dev/null
@@ -1,28 +0,0 @@
-// Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
-package com.yahoo.vespa.model;
-
-import java.util.logging.Formatter;
-import java.util.logging.LogRecord;
-
-/**
- * A log formatter that returns a plain log message only with level, not
- * including timestamp and method (as java.util.logging.SimpleFormatter).
- * See bug #1789867.
- *
- * @author gjoranv
- */
-public class PlainFormatter extends Formatter {
-
-    public PlainFormatter() {
-        super();
-    }
-
-    public String format(LogRecord record) {
-        StringBuffer sb = new StringBuffer();
-
-        sb.append(record.getLevel().getName()).append(": ");
-        sb.append(formatMessage(record)).append("\n");
-
-        return sb.toString();
-    }
-}
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/PortsMeta.java b/config-model/src/main/java/com/yahoo/vespa/model/PortsMeta.java
index ea2151f9976..a0b3cc7294b 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/PortsMeta.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/PortsMeta.java
@@ -7,11 +7,12 @@ import java.util.LinkedList;
 import java.util.List;
 
 /**
- * Track metainformation about the ports of a service.
+ * Track meta information about the ports of a service.
  *
  * @author Vidar Larsen
  */
 public class PortsMeta implements Serializable {
+
     /** A list of all ports. The list elements are lists of strings. */
     private List<LinkedList<String>> ports;
 
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/builder/VespaModelBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/builder/VespaModelBuilder.java
index 75e9caefbd5..bcf523e1c99 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/builder/VespaModelBuilder.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/builder/VespaModelBuilder.java
@@ -27,4 +27,5 @@ public abstract class VespaModelBuilder {
      * @param configModelRepo a {@link com.yahoo.config.model.ConfigModelRepo instance}
      */
     public abstract void postProc(AbstractConfigProducer producerRoot, ConfigModelRepo configModelRepo);
+
 }
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomClientsBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomClientsBuilder.java
index 876017e16bc..f1829a1d718 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomClientsBuilder.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomClientsBuilder.java
@@ -35,4 +35,5 @@ public class DomClientsBuilder extends LegacyConfigModelBuilder<Clients> {
             throw new IllegalArgumentException("Version '" + version + "' of 'clients' not supported.");
         }
     }
+
 }
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomV20ClientsBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomV20ClientsBuilder.java
index ff37c0a95fd..b4070c67ae1 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomV20ClientsBuilder.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/DomV20ClientsBuilder.java
@@ -1,40 +1,17 @@
 // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.model.builder.xml.dom;
 
-import com.yahoo.component.ComponentId;
-import com.yahoo.component.ComponentSpecification;
-import com.yahoo.component.chain.Phase;
-import com.yahoo.component.chain.dependencies.Dependencies;
-import com.yahoo.component.chain.model.ChainSpecification;
-import com.yahoo.component.chain.model.ChainedComponentModel;
-import com.yahoo.config.model.ConfigModelUtils;
 import com.yahoo.vespa.config.content.spooler.SpoolerConfig;
 import com.yahoo.config.model.producer.AbstractConfigProducer;
-import com.yahoo.container.bundle.BundleInstantiationSpecification;
-import com.yahoo.osgi.provider.model.ComponentModel;
 import com.yahoo.text.XML;
 import com.yahoo.vespa.defaults.Defaults;
 import com.yahoo.vespa.model.SimpleConfigProducer;
 import com.yahoo.vespa.model.builder.xml.dom.VespaDomBuilder.DomConfigProducerBuilder;
-import com.yahoo.vespa.model.builder.xml.dom.chains.docproc.DomDocprocChainsBuilder;
 import com.yahoo.vespa.model.clients.Clients;
-import com.yahoo.vespa.model.clients.HttpGatewayOwner;
 import com.yahoo.vespa.model.clients.VespaSpoolMaster;
 import com.yahoo.vespa.model.clients.VespaSpooler;
 import com.yahoo.vespa.model.clients.VespaSpoolerProducer;
 import com.yahoo.vespa.model.clients.VespaSpoolerService;
-import com.yahoo.vespa.model.container.Container;
-import com.yahoo.vespa.model.container.ContainerCluster;
-import com.yahoo.vespa.model.container.component.Handler;
-import com.yahoo.vespa.model.container.component.chain.ProcessingHandler;
-import com.yahoo.vespa.model.container.docproc.ContainerDocproc;
-import com.yahoo.vespa.model.container.docproc.DocprocChains;
-import com.yahoo.vespa.model.container.search.ContainerHttpGateway;
-import com.yahoo.vespa.model.container.search.ContainerSearch;
-import com.yahoo.vespa.model.container.search.searchchain.SearchChain;
-import com.yahoo.vespa.model.container.search.searchchain.SearchChains;
-import com.yahoo.vespa.model.container.search.searchchain.Searcher;
-import com.yahoo.vespa.model.container.xml.ContainerModelBuilder;
 import com.yahoo.vespaclient.config.FeederConfig;
 import org.w3c.dom.Element;
 import org.w3c.dom.Node;
@@ -43,9 +20,6 @@ import org.w3c.dom.NodeList;
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Set;
-import java.util.TreeSet;
-import java.util.logging.Level;
 
 /**
  * Builds the Clients plugin