From 7dfbfa34e7a18ad76215435950ee36d772405bf5 Mon Sep 17 00:00:00 2001 From: Dmitry Kryukov Date: Mon, 31 Mar 2025 11:05:07 +0300 Subject: [PATCH 1/2] [optimization] commented out unused stringbuilder --- .../main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java index 57a100f566..0810e2d4d5 100644 --- a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java +++ b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java @@ -802,10 +802,10 @@ public List getDiagnostics() { public DAGReport getReport() { readLock.lock(); try { - StringBuilder diagsb = new StringBuilder(); + /*StringBuilder diagsb = new StringBuilder(); for (String s : getDiagnostics()) { diagsb.append(s).append("\n"); - } + }*/ if (getInternalState() == DAGState.NEW) { /* From 6b70dd9fd4f13ca652a95774e86dde70dc525c46 Mon Sep 17 00:00:00 2001 From: Dmitry Kryukov Date: Mon, 31 Mar 2025 23:08:28 +0300 Subject: [PATCH 2/2] [optimization] removed unused stringbuilder --- .../main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java index 0810e2d4d5..e20351d46e 100644 --- a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java +++ b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/DAGImpl.java @@ -802,11 +802,6 @@ public List getDiagnostics() { public DAGReport getReport() { readLock.lock(); try { - /*StringBuilder diagsb = new StringBuilder(); - for (String s : getDiagnostics()) { - diagsb.append(s).append("\n"); - }*/ - if (getInternalState() == DAGState.NEW) { /* return MRBuilderUtils.newJobReport(dagId, dagName, username, state,