Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HDFS-8659. Block scanner INFO message is spamming logs. Contributed by Yongjun Zhang. 

(cherry picked from commit 43a1288fff)
This commit is contained in:
Yongjun Zhang 2015-06-29 14:35:39 -07:00
parent 484a6c0f01
commit 11bdcb754a
3 changed files with 7 additions and 5 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

@ -652,6 +652,8 @@ Release 2.7.2 - UNRELEASED
IMPROVEMENTS IMPROVEMENTS
HDFS-8659. Block scanner INFO message is spamming logs. (Yongjun Zhang)
OPTIMIZATIONS OPTIMIZATIONS
BUG FIXES BUG FIXES

2
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/BlockScanner.java
View File

@ -317,7 +317,7 @@ public class BlockScanner {
*/ */
synchronized void markSuspectBlock(String storageId, ExtendedBlock block) { synchronized void markSuspectBlock(String storageId, ExtendedBlock block) {
if (!isEnabled()) { if (!isEnabled()) {
LOG.info("Not scanning suspicious block {} on {}, because the block " + LOG.debug("Not scanning suspicious block {} on {}, because the block " +
"scanner is disabled.", block, storageId); "scanner is disabled.", block, storageId);
return; return;
} }

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

@ -656,24 +656,24 @@ public class VolumeScanner extends Thread {
public synchronized void markSuspectBlock(ExtendedBlock block) { public synchronized void markSuspectBlock(ExtendedBlock block) {
if (stopping) { if (stopping) {
LOG.info("{}: Not scheduling suspect block {} for " + LOG.debug("{}: Not scheduling suspect block {} for " +
"rescanning, because this volume scanner is stopping.", this, block); "rescanning, because this volume scanner is stopping.", this, block);
return; return;
} }
Boolean recent = recentSuspectBlocks.getIfPresent(block); Boolean recent = recentSuspectBlocks.getIfPresent(block);
if (recent != null) { if (recent != null) {
LOG.info("{}: Not scheduling suspect block {} for " + LOG.debug("{}: Not scheduling suspect block {} for " +
"rescanning, because we rescanned it recently.", this, block); "rescanning, because we rescanned it recently.", this, block);
return; return;
} }
if (suspectBlocks.contains(block)) { if (suspectBlocks.contains(block)) {
LOG.info("{}: suspect block {} is already queued for " + LOG.debug("{}: suspect block {} is already queued for " +
"rescanning.", this, block); "rescanning.", this, block);
return; return;
} }
suspectBlocks.add(block); suspectBlocks.add(block);
recentSuspectBlocks.put(block, true); recentSuspectBlocks.put(block, true);
LOG.info("{}: Scheduling suspect block {} for rescanning.", this, block); LOG.debug("{}: Scheduling suspect block {} for rescanning.", this, block);
notify(); // wake scanner thread. notify(); // wake scanner thread.
} }