[
https://issues.apache.org/jira/browse/YARN-10647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17288446#comment-17288446
]
Qi Zhu commented on YARN-10647:
-------------------------------
[~caozhiqiang] [~ebadger]
Could you help review this?
The TestRMNodeLabelsManager failed after -YARN-10501.-
The deactivateNode should make sure the nm.labels null, to remove related.
{code:java}
public void deactivateNode(NodeId nodeId) {
writeLock.lock();
try {
// save if we have a node before
Map<String, Host> before = cloneNodeMap(ImmutableSet.of(nodeId));
Node nm = getNMInNodeSet(nodeId);
if (null != nm) {
if (null == nm.labels) {
// When node deactivated, remove the nm from node collection if no
// labels explicitly set for this particular nm
// Save labels first, we need to remove label->nodes relation later
Set<String> savedNodeLabels = getLabelsOnNode(nodeId);
// Remove this node in nodes collection
nodeCollections.get(nodeId.getHost()).nms.remove(nodeId);
// Remove this node in labels->node
removeNodeFromLabels(nodeId, savedNodeLabels);
} else {
// set nm is not running, and its resource = 0
nm.running = false;
nm.resource = Resource.newInstance(0, 0);
}
}
// get the node after edition
Map<String, Host> after = cloneNodeMap(ImmutableSet.of(nodeId));
updateResourceMappings(before, after);
} finally {
writeLock.unlock();
}
}
{code}
Thanks.
> Fix TestRMNodeLabelsManager failed after YARN-10501.
> ----------------------------------------------------
>
> Key: YARN-10647
> URL: https://issues.apache.org/jira/browse/YARN-10647
> Project: Hadoop YARN
> Issue Type: Bug
> Reporter: Qi Zhu
> Assignee: Qi Zhu
> Priority: Major
> Attachments: YARN-10647.001.patch
>
>
> YARN-10501 introduced the TestRMNodeLabelsManager test failed.
> Because the deactive for label, should be null.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
