HDFS-11827. NPE is thrown when log level changed in BlockPlacementPolicyDefault#chooseRandom() method. Contributed by xupeng.
This commit is contained in:
parent
82ea3f4545
commit
0e95ff78c3
|
@ -718,7 +718,7 @@ public class BlockPlacementPolicyDefault extends BlockPlacementPolicy {
|
||||||
}
|
}
|
||||||
Preconditions.checkState(excludedNodes.add(chosenNode), "chosenNode "
|
Preconditions.checkState(excludedNodes.add(chosenNode), "chosenNode "
|
||||||
+ chosenNode + " is already in excludedNodes " + excludedNodes);
|
+ chosenNode + " is already in excludedNodes " + excludedNodes);
|
||||||
if (LOG.isDebugEnabled()) {
|
if (LOG.isDebugEnabled() && builder != null) {
|
||||||
builder.append("\nNode ").append(NodeBase.getPath(chosenNode))
|
builder.append("\nNode ").append(NodeBase.getPath(chosenNode))
|
||||||
.append(" [");
|
.append(" [");
|
||||||
}
|
}
|
||||||
|
@ -747,7 +747,7 @@ public class BlockPlacementPolicyDefault extends BlockPlacementPolicy {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (LOG.isDebugEnabled()) {
|
if (LOG.isDebugEnabled() && builder != null) {
|
||||||
builder.append("\n]");
|
builder.append("\n]");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue