From 2d929bda295e05d4b7a3194a7dc311944b662661 Mon Sep 17 00:00:00 2001
From: "zengqiang.xu" <zanderxu@apache.org>
Date: Sat, 11 May 2024 15:08:39 +0800
Subject: [PATCH] HDFS-17520. [BugFix] TestDFSAdmin.testAllDatanodesReconfig
 and TestDFSAdmin.testDecommissionDataNodesReconfig failed

---
 .../java/org/apache/hadoop/hdfs/tools/DFSAdmin.java   | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java
index b57c146023c69..8e57bcb3beb7a 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java
@@ -2095,22 +2095,20 @@ int getReconfigurationStatus(final String nodeType, final String address, final
     if (errMsg != null) {
       err.println(errMsg);
       return 1;
-    } else {
-      out.print(outMsg);
     }
 
     if (status != null) {
       if (!status.hasTask()) {
-        out.println("no task was found.");
+        out.println(outMsg + "no task was found.");
         return 0;
       }
-      out.print("started at " + new Date(status.getStartTime()));
+      String startMsg = outMsg + "started at " + new Date(status.getStartTime());
       if (!status.stopped()) {
-        out.println(" and is still running.");
+        out.println(startMsg + " and is still running.");
         return 0;
       }
 
-      out.println(" and finished at "
+      out.println(startMsg + " and finished at "
           + new Date(status.getEndTime()).toString() + ".");
       if (status.getStatus() == null) {
         // Nothing to report.
@@ -2133,6 +2131,7 @@ int getReconfigurationStatus(final String nodeType, final String address, final
         }
       }
     } else {
+      out.println(outMsg);
       return 1;
     }