merge -r 1352637:1352638 from trunk. FIXES: MAPREDUCE-4295
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1352639 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
2d734774a1
commit
22d23a4d0f
|
@ -482,6 +482,8 @@ Release 0.23.3 - UNRELEASED
|
||||||
|
|
||||||
MAPREDUCE-4320. gridmix mainClass wrong in pom.xml (tgraves)
|
MAPREDUCE-4320. gridmix mainClass wrong in pom.xml (tgraves)
|
||||||
|
|
||||||
|
MAPREDUCE-4295. RM crashes due to DNS issue (tgraves)
|
||||||
|
|
||||||
Release 0.23.2 - UNRELEASED
|
Release 0.23.2 - UNRELEASED
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
|
@ -1180,9 +1180,16 @@ public class LeafQueue implements CSQueue {
|
||||||
if (UserGroupInformation.isSecurityEnabled()) {
|
if (UserGroupInformation.isSecurityEnabled()) {
|
||||||
ContainerTokenIdentifier tokenIdentifier = new ContainerTokenIdentifier(
|
ContainerTokenIdentifier tokenIdentifier = new ContainerTokenIdentifier(
|
||||||
containerId, nodeId.toString(), capability);
|
containerId, nodeId.toString(), capability);
|
||||||
|
try {
|
||||||
containerToken = BuilderUtils.newContainerToken(nodeId, ByteBuffer
|
containerToken = BuilderUtils.newContainerToken(nodeId, ByteBuffer
|
||||||
.wrap(containerTokenSecretManager
|
.wrap(containerTokenSecretManager
|
||||||
.createPassword(tokenIdentifier)), tokenIdentifier);
|
.createPassword(tokenIdentifier)), tokenIdentifier);
|
||||||
|
} catch (IllegalArgumentException e) {
|
||||||
|
// this could be because DNS is down - in which case we just want
|
||||||
|
// to retry and not bring RM down
|
||||||
|
LOG.error("Error trying to create new container", e);
|
||||||
|
return null;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Create the container
|
// Create the container
|
||||||
|
@ -1212,6 +1219,11 @@ public class LeafQueue implements CSQueue {
|
||||||
Container container =
|
Container container =
|
||||||
getContainer(rmContainer, application, node, capability, priority);
|
getContainer(rmContainer, application, node, capability, priority);
|
||||||
|
|
||||||
|
// something went wrong getting/creating the container
|
||||||
|
if (container == null) {
|
||||||
|
return Resources.none();
|
||||||
|
}
|
||||||
|
|
||||||
// Can we allocate a container on this node?
|
// Can we allocate a container on this node?
|
||||||
int availableContainers =
|
int availableContainers =
|
||||||
available.getMemory() / capability.getMemory();
|
available.getMemory() / capability.getMemory();
|
||||||
|
|
Loading…
Reference in New Issue