diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-03-13 10:41:24 +0100 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2020-03-13 10:41:24 +0100 |
commit | 7e63a481a108837843953029c6f94335672ac170 (patch) | |
tree | 9c2a03b6f7a7f582317dbb2aa3ca548f9b8437ff | |
parent | 76e8ead456dfa160eec758d079de40fd95e10567 (diff) |
Fix StringBuilder usage based on review feedback
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java index b10fb7601d7..b4246171277 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java @@ -158,7 +158,8 @@ public class RankSetupValidator extends Validator { } private void validateFail(String output, SearchCluster sc, String sdName, DeployLogger deployLogger) { - StringBuilder errMsg = new StringBuilder("For search cluster '" + sc.getClusterName() + "', search definition '" + sdName + "': error in rank setup. Details:\n"); + StringBuilder errMsg = new StringBuilder("For search cluster '").append(sc.getClusterName()).append("', ") + .append("search definition '").append(sdName).append("': error in rank setup. Details:\n"); for (String line : output.split("\n")) { // Remove debug lines from start script if (line.startsWith("debug\t")) continue; @@ -170,7 +171,7 @@ public class RankSetupValidator extends Validator { } } if (ignoreValidationErrors) { - deployLogger.log(LogLevel.WARNING, errMsg + "(Continuing since ignoreValidationErrors flag is set.)"); + deployLogger.log(LogLevel.WARNING, errMsg.append("(Continuing since ignoreValidationErrors flag is set.)").toString()); } else { throw new IllegalArgumentException(errMsg.toString()); } |