commit | b85ee0d75d450f1c695cec1bebac627f66a97c98 | [log] [tgz] |
---|---|---|
author | Vidura Mudalige <vidura.mudalige@kohls.com> | Thu Sep 27 21:40:13 2018 +0530 |
committer | Akira Ajisaka <aajisaka@apache.org> | Sun Oct 07 00:53:53 2018 +0900 |
tree | 89d3b5c5e8b4d0f1d9e803ecc6c2efff654265a7 | |
parent | cdf5d58364afbb58b3ae49670a7b179d6c5a0ba7 [diff] |
YARN-8819. Fix findbugs warnings in YarnServiceUtils Signed-off-by: Akira Ajisaka <aajisaka@apache.org>
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceUtils.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceUtils.java index 26402da..d69840a 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceUtils.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceUtils.java
@@ -115,15 +115,20 @@ private static String getComponentArrayJson(String componentName, int count, String endpointSuffix) { String component = "\\\"" + componentName + "\\\":"; - String array = "["; + StringBuilder array = new StringBuilder(); + array.append("["); for (int i = 0; i < count; i++) { - array = array + "\\\"" + componentName + "-" + i - + endpointSuffix + "\\\""; + array.append("\\\""); + array.append(componentName); + array.append("-"); + array.append(i); + array.append(endpointSuffix); + array.append("\\\""); if (i != count - 1) { - array = array + ","; + array.append(","); } } - array = array + "]"; - return component + array; + array.append("]"); + return component + array.toString(); } }