From 0862204e36bc4e47be69480c63bb2980bda88e7a Mon Sep 17 00:00:00 2001 From: Jon Bratseth Date: Fri, 11 Oct 2019 10:05:04 +0200 Subject: Nonfunctional changes only --- .../src/main/java/com/yahoo/config/provision/Rotation.java | 8 ++++++-- .../com/yahoo/vespa/config/server/session/PrepareParams.java | 9 +++++---- .../yahoo/vespa/config/server/session/SessionPreparer.java | 3 --- .../java/com/yahoo/search/grouping/request/StringBucket.java | 12 ++++++------ .../java/com/yahoo/vespa/hosted/controller/Instance.java | 12 ++++-------- vespalog/src/main/java/com/yahoo/log/LogSetup.java | 4 +++- vespalog/src/main/java/com/yahoo/log/VespaFormat.java | 3 +-- vespalog/src/main/java/com/yahoo/log/VespaFormatter.java | 5 +---- vespalog/src/main/java/com/yahoo/log/VespaLogHandler.java | 5 ++--- 9 files changed, 28 insertions(+), 33 deletions(-) diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/Rotation.java b/config-provisioning/src/main/java/com/yahoo/config/provision/Rotation.java index 7eaebbe848a..75b7d790eba 100644 --- a/config-provisioning/src/main/java/com/yahoo/config/provision/Rotation.java +++ b/config-provisioning/src/main/java/com/yahoo/config/provision/Rotation.java @@ -1,11 +1,10 @@ // Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.config.provision; - import java.util.Objects; /** - * A Brooklyn rotation, e.g. rotation-042.vespa.a02.yahoodns.net. + * A rotation (virtual endpoint). */ public class Rotation { @@ -36,4 +35,9 @@ public class Rotation { return id.hashCode(); } + @Override + public String toString() { + return "rotation '" + id + "'"; + } + } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/PrepareParams.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/PrepareParams.java index 5bf70c55f9e..a21e12a5d42 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/PrepareParams.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/PrepareParams.java @@ -49,8 +49,9 @@ public final class PrepareParams { private final Optional tlsSecretsKeyName; private PrepareParams(ApplicationId applicationId, TimeoutBudget timeoutBudget, boolean ignoreValidationErrors, - boolean dryRun, boolean verbose, boolean isBootstrap, Optional vespaVersion, Set rotations, - List containerEndpoints, Optional tlsSecretsKeyName) { + boolean dryRun, boolean verbose, boolean isBootstrap, Optional vespaVersion, + Set rotations, List containerEndpoints, + Optional tlsSecretsKeyName) { this.timeoutBudget = timeoutBudget; this.applicationId = applicationId; this.ignoreValidationErrors = ignoreValidationErrors; @@ -140,8 +141,8 @@ public final class PrepareParams { if (serialized == null) return this; Slime slime = SlimeUtils.jsonToSlime(serialized); containerEndpoints = ContainerEndpointSerializer.endpointListFromSlime(slime); - return this; - } + return this; + } public Builder tlsSecretsKeyName(String tlsSecretsKeyName) { this.tlsSecretsKeyName = Optional.ofNullable(tlsSecretsKeyName) diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java index 3ec74a4c3eb..872936f7f34 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java @@ -72,7 +72,6 @@ public class SessionPreparer { private final ConfigDefinitionRepo configDefinitionRepo; private final Curator curator; private final Zone zone; - private final FlagSource flagSource; private final SecretStore secretStore; @Inject @@ -84,7 +83,6 @@ public class SessionPreparer { ConfigDefinitionRepo configDefinitionRepo, Curator curator, Zone zone, - FlagSource flagSource, SecretStore secretStore) { this.modelFactoryRegistry = modelFactoryRegistry; this.fileDistributionFactory = fileDistributionFactory; @@ -94,7 +92,6 @@ public class SessionPreparer { this.configDefinitionRepo = configDefinitionRepo; this.curator = curator; this.zone = zone; - this.flagSource = flagSource; this.secretStore = secretStore; } diff --git a/container-search/src/main/java/com/yahoo/search/grouping/request/StringBucket.java b/container-search/src/main/java/com/yahoo/search/grouping/request/StringBucket.java index 5558f1443db..2718abe7ac1 100644 --- a/container-search/src/main/java/com/yahoo/search/grouping/request/StringBucket.java +++ b/container-search/src/main/java/com/yahoo/search/grouping/request/StringBucket.java @@ -11,8 +11,8 @@ public class StringBucket extends BucketValue { /** * Get the next distinct value. * - * @param value The base value. - * @return the next value. + * @param value the base value + * @return the next value */ public static StringValue nextValue(StringValue value) { return new StringValue(value.getValue() + " "); @@ -21,8 +21,8 @@ public class StringBucket extends BucketValue { /** * Constructs a new instance of this class. * - * @param from The from-value to assign to this. - * @param to The to-value to assign to this. + * @param from the from-value to assign to this + * @param to the to-value to assign to this */ public StringBucket(String from, String to) { super(null, null, new StringValue(from), new StringValue(to)); @@ -31,8 +31,8 @@ public class StringBucket extends BucketValue { /** * Constructs a new instance of this class. * - * @param from The from-value to assign to this. - * @param to The to-value to assign to this. + * @param from the from-value to assign to this + * @param to the to-value to assign to this */ public StringBucket(ConstantValue from, ConstantValue to) { super(null, null, from, to); diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Instance.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Instance.java index 627cde28fd0..90fa51fb8f8 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Instance.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Instance.java @@ -117,18 +117,15 @@ public class Instance { } public Instance withoutDeploymentJob(JobType jobType) { - return new Instance(id, deployments.values(), deploymentJobs.without(jobType), - rotations, rotationStatus); + return new Instance(id, deployments.values(), deploymentJobs.without(jobType), rotations, rotationStatus); } public Instance with(List assignedRotations) { - return new Instance(id, deployments.values(), deploymentJobs, - assignedRotations, rotationStatus); + return new Instance(id, deployments.values(), deploymentJobs, assignedRotations, rotationStatus); } public Instance with(RotationStatus rotationStatus) { - return new Instance(id, deployments.values(), deploymentJobs, - rotations, rotationStatus); + return new Instance(id, deployments.values(), deploymentJobs, rotations, rotationStatus); } private Instance with(Deployment deployment) { @@ -138,8 +135,7 @@ public class Instance { } private Instance with(Map deployments) { - return new Instance(id, deployments.values(), deploymentJobs, - rotations, rotationStatus); + return new Instance(id, deployments.values(), deploymentJobs, rotations, rotationStatus); } public ApplicationId id() { return id; } diff --git a/vespalog/src/main/java/com/yahoo/log/LogSetup.java b/vespalog/src/main/java/com/yahoo/log/LogSetup.java index 12b1411b17e..1bdefaa52fd 100644 --- a/vespalog/src/main/java/com/yahoo/log/LogSetup.java +++ b/vespalog/src/main/java/com/yahoo/log/LogSetup.java @@ -82,7 +82,7 @@ public class LogSetup { String logControlDir = System.getProperty("vespa.log.control.dir"); String logControlFile = System.getProperty("vespa.log.control.file"); if (programName == null || programName.equals("")) { - throw new RuntimeException("invalid programName: "+programName); + throw new RuntimeException("invalid programName: " + programName); } // then try environment values @@ -165,6 +165,7 @@ public class LogSetup { * For other log records, isLoggable returns true */ static class ZooKeeperFilter implements Filter { + private FileHandler fileHandler; ZooKeeperFilter(String logFile) { @@ -180,6 +181,7 @@ public class LogSetup { /** * Return true if loggable (ordinary log record), returns false if this filter * logs it itself + * * @param record a #{@link LogRecord} * @return true if loggable, false otherwise */ diff --git a/vespalog/src/main/java/com/yahoo/log/VespaFormat.java b/vespalog/src/main/java/com/yahoo/log/VespaFormat.java index d0dc5607af5..90e16f6408e 100644 --- a/vespalog/src/main/java/com/yahoo/log/VespaFormat.java +++ b/vespalog/src/main/java/com/yahoo/log/VespaFormat.java @@ -48,8 +48,7 @@ public class VespaFormat { * value was "(empty)". * * @param s String that might need escaping - * @return Returns escaped string - * + * @return returns the escaped string */ public static String escape (String s) { if (s == null) { diff --git a/vespalog/src/main/java/com/yahoo/log/VespaFormatter.java b/vespalog/src/main/java/com/yahoo/log/VespaFormatter.java index 482d7049079..314e0ae49e6 100644 --- a/vespalog/src/main/java/com/yahoo/log/VespaFormatter.java +++ b/vespalog/src/main/java/com/yahoo/log/VespaFormatter.java @@ -92,9 +92,7 @@ public class VespaFormatter extends SimpleFormatter { public String format(LogRecord r) { StringBuilder sbuf = new StringBuilder(300); // initial guess - String levelName = LogLevel.getVespaLogLevel(r.getLevel()) - .toString() - .toLowerCase(); + String levelName = LogLevel.getVespaLogLevel(r.getLevel()).toString().toLowerCase(); String component = r.getLoggerName(); @@ -130,7 +128,6 @@ public class VespaFormatter extends SimpleFormatter { } appendException(r.getThrown(), sbuf); - sbuf.append("\n"); return sbuf.toString(); } diff --git a/vespalog/src/main/java/com/yahoo/log/VespaLogHandler.java b/vespalog/src/main/java/com/yahoo/log/VespaLogHandler.java index 7eede7e1e35..331780f226b 100644 --- a/vespalog/src/main/java/com/yahoo/log/VespaLogHandler.java +++ b/vespalog/src/main/java/com/yahoo/log/VespaLogHandler.java @@ -7,7 +7,7 @@ import java.util.logging.LogRecord; import java.util.logging.StreamHandler; /** - * @author Bjorn Borud + * @author Bjorn Borud * @author arnej27959 */ class VespaLogHandler extends StreamHandler { @@ -32,8 +32,7 @@ class VespaLogHandler extends StreamHandler { * */ VespaLogHandler(LogTarget logTarget, - LevelControllerRepo levelControllerRepo, String serviceName, - String applicationPrefix) { + LevelControllerRepo levelControllerRepo, String serviceName, String applicationPrefix) { this.logTarget = logTarget; this.serviceName = serviceName; this.appPrefix = applicationPrefix; -- cgit v1.2.3