Browse Source

HDFS-15895 : Remove redundant String#format in DFSAdmin#printOpenFiles (#2772)

(cherry picked from commit 9c43b60348bea7f89f7c0fc86f2813fdf2d1929f)
Viraj Jasani 4 years ago
parent
commit
503101c1f1

+ 4 - 4
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java

@@ -1001,14 +1001,14 @@ public class DFSAdmin extends FsShell {
 
 
   private void printOpenFiles(RemoteIterator<OpenFileEntry> openFilesIterator)
   private void printOpenFiles(RemoteIterator<OpenFileEntry> openFilesIterator)
       throws IOException {
       throws IOException {
-    System.out.println(String.format("%-20s\t%-20s\t%s", "Client Host",
-          "Client Name", "Open File Path"));
+    System.out.printf("%-20s\t%-20s\t%s%n", "Client Host",
+        "Client Name", "Open File Path");
     while (openFilesIterator.hasNext()) {
     while (openFilesIterator.hasNext()) {
       OpenFileEntry openFileEntry = openFilesIterator.next();
       OpenFileEntry openFileEntry = openFilesIterator.next();
-      System.out.println(String.format("%-20s\t%-20s\t%20s",
+      System.out.printf("%-20s\t%-20s\t%20s%n",
           openFileEntry.getClientMachine(),
           openFileEntry.getClientMachine(),
           openFileEntry.getClientName(),
           openFileEntry.getClientName(),
-          openFileEntry.getFilePath()));
+          openFileEntry.getFilePath());
     }
     }
   }
   }