HDFS-8659. Block scanner INFO message is spamming logs. Contributed by Yongjun Zhang.
This commit is contained in:
parent
34ee0b9b47
commit
43a1288fff
|
@ -985,6 +985,8 @@ Release 2.7.2 - UNRELEASED
|
|||
|
||||
IMPROVEMENTS
|
||||
|
||||
HDFS-8659. Block scanner INFO message is spamming logs. (Yongjun Zhang)
|
||||
|
||||
OPTIMIZATIONS
|
||||
|
||||
BUG FIXES
|
||||
|
|
|
@ -317,7 +317,7 @@ public class BlockScanner {
|
|||
*/
|
||||
synchronized void markSuspectBlock(String storageId, ExtendedBlock block) {
|
||||
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);
|
||||
return;
|
||||
}
|
||||
|
|
|
@ -656,24 +656,24 @@ public class VolumeScanner extends Thread {
|
|||
|
||||
public synchronized void markSuspectBlock(ExtendedBlock block) {
|
||||
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);
|
||||
return;
|
||||
}
|
||||
Boolean recent = recentSuspectBlocks.getIfPresent(block);
|
||||
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);
|
||||
return;
|
||||
}
|
||||
if (suspectBlocks.contains(block)) {
|
||||
LOG.info("{}: suspect block {} is already queued for " +
|
||||
LOG.debug("{}: suspect block {} is already queued for " +
|
||||
"rescanning.", this, block);
|
||||
return;
|
||||
}
|
||||
suspectBlocks.add(block);
|
||||
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.
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue