HDFS-10620. StringBuilder created and appended even if logging is disabled. Contributed by Staffan Friberg.

This commit is contained in:
Akira Ajisaka 2017-02-25 04:15:43 +09:00
parent 19c5b3d622
commit bd3c2a2ee9
1 changed files with 6 additions and 3 deletions

View File

@ -1303,16 +1303,19 @@ public class BlockManager implements BlockStatsMXBean {
if (!isPopulatingReplQueues()) {
return;
}
StringBuilder datanodes = new StringBuilder();
StringBuilder datanodes = blockLog.isDebugEnabled()
? new StringBuilder() : null;
for (DatanodeStorageInfo storage : blocksMap.getStorages(b)) {
if (storage.getState() != State.NORMAL) {
continue;
}
final DatanodeDescriptor node = storage.getDatanodeDescriptor();
invalidateBlocks.add(b, node, false);
datanodes.append(node).append(" ");
if (datanodes != null) {
datanodes.append(node).append(" ");
}
}
if (datanodes.length() != 0) {
if (datanodes != null && datanodes.length() != 0) {
blockLog.debug("BLOCK* addToInvalidates: {} {}", b, datanodes);
}
}