big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDFS-9210. Fix some misuse of %n in VolumeScanner#printStats. Contributed by Xiaoyu Yao.

Browse Source
This commit is contained in:
Xiaoyu Yao 2015-10-14 13:22:58 -07:00
parent ba3c197878
commit 3d50855952
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
View File

@ -2060,6 +2060,8 @@ Release 2.8.0 - UNRELEASED
HDFS-9187. Fix null pointer error in Globber when FS was not constructed HDFS-9187. Fix null pointer error in Globber when FS was not constructed
via FileSystem#createFileSystem (cmccabe) via FileSystem#createFileSystem (cmccabe)
HDFS-9210. Fix some misuse of %n in VolumeScanner#printStats. (xyao)
Release 2.7.2 - UNRELEASED Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

7
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/VolumeScanner.java
View File

@ -216,9 +216,8 @@ private static double positiveMsToHours(long ms) {
} }
public void printStats(StringBuilder p) { public void printStats(StringBuilder p) {
p.append("Block scanner information for volume " + p.append(String.format("Block scanner information for volume %s with base" +
volume.getStorageID() + " with base path " + volume.getBasePath() + " path %s%n" + volume.getStorageID(), volume.getBasePath()));
"%n");
synchronized (stats) { synchronized (stats) {
p.append(String.format("Bytes verified in last hour : %57d%n", p.append(String.format("Bytes verified in last hour : %57d%n",
stats.bytesScannedInPastHour)); stats.bytesScannedInPastHour));
@ -245,7 +244,7 @@ public void printStats(StringBuilder p) {
stats.lastBlockScanned.toString()))); stats.lastBlockScanned.toString())));
p.append(String.format("More blocks to scan in period : %57s%n", p.append(String.format("More blocks to scan in period : %57s%n",
!stats.eof)); !stats.eof));
p.append("%n"); p.append(String.format("%n"));
} }
} }