From efc36ebdba0b5e2245e9257e02229b9dcddb2dfd Mon Sep 17 00:00:00 2001 From: jiaoqingbo <1178404354@qq.com> Date: Tue, 17 Oct 2023 16:51:29 +0800 Subject: [PATCH] =?UTF-8?q?[CELEBORN-1043]=20Convert=20variable=20?= =?UTF-8?q?=E2=80=98metric=E2=80=99=20from=20String=20to=20StringBuilder?= =?UTF-8?q?=20in=20toMetric=20method?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ### What changes were proposed in this pull request? As Title ### Why are the changes needed? As Title ### Does this PR introduce _any_ user-facing change? NO ### How was this patch tested? PASS GA Closes #1996 from jiaoqingbo/1043. Authored-by: jiaoqingbo <1178404354@qq.com> Signed-off-by: zky.zhoukeyong --- .../java/org/apache/celeborn/common/meta/DiskStatus.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/common/src/main/java/org/apache/celeborn/common/meta/DiskStatus.java b/common/src/main/java/org/apache/celeborn/common/meta/DiskStatus.java index edf313271..6180f64f3 100644 --- a/common/src/main/java/org/apache/celeborn/common/meta/DiskStatus.java +++ b/common/src/main/java/org/apache/celeborn/common/meta/DiskStatus.java @@ -37,14 +37,14 @@ public enum DiskStatus { public final String toMetric() { String[] fragments = this.name().split("_"); - String metric = ""; + StringBuilder metric = new StringBuilder(); for (String fragment : fragments) { int len = fragment.length(); if (len >= 1) { - metric += fragment.substring(0, 1).toUpperCase(); - metric += fragment.substring(1, len).toLowerCase(); + metric.append(fragment.substring(0, 1).toUpperCase()); + metric.append(fragment.substring(1, len).toLowerCase()); } } - return metric; + return metric.toString(); } }