HDFS-16074. Remove an expensive debug string concatenation

jojochuang · jojochuang · commit c8ba1a525edb · 2021-06-16T14:41:32.000+08:00
Change-Id: I6c5738753d6aa759e0916d2f052c974c23f618bf
diff --git a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
@@ -483,9 +483,10 @@ private synchronized void writeChunkPrepare(int buflen,
       currentPacket = createPacket(packetSize, chunksPerPacket, getStreamer()
           .getBytesCurBlock(), getStreamer().getAndIncCurrentSeqno(), false);
       DFSClient.LOG.debug("WriteChunk allocating new packet seqno={},"
-              + " src={}, packetSize={}, chunksPerPacket={}, bytesCurBlock={}",
+              + " src={}, packetSize={}, chunksPerPacket={}, bytesCurBlock={},"
+              + " output stream={}",
           currentPacket.getSeqno(), src, packetSize, chunksPerPacket,
-          getStreamer().getBytesCurBlock() + ", " + this);
+          getStreamer().getBytesCurBlock(), this);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -483,9 +483,10 @@ private synchronized void writeChunkPrepare(int buflen,`
`483`	`483`	`currentPacket = createPacket(packetSize, chunksPerPacket, getStreamer()`
`484`	`484`	`.getBytesCurBlock(), getStreamer().getAndIncCurrentSeqno(), false);`
`485`	`485`	`DFSClient.LOG.debug("WriteChunk allocating new packet seqno={},"`
`486`		`- + " src={}, packetSize={}, chunksPerPacket={}, bytesCurBlock={}",`
	`486`	`+ + " src={}, packetSize={}, chunksPerPacket={}, bytesCurBlock={},"`
	`487`	`+ + " output stream={}",`
`487`	`488`	`currentPacket.getSeqno(), src, packetSize, chunksPerPacket,`
`488`		`- getStreamer().getBytesCurBlock() + ", " + this);`
	`489`	`+ getStreamer().getBytesCurBlock(), this);`
`489`	`490`	`}`
`490`	`491`	`}`
`491`	`492`