[YARN-4386] refreshNodesGracefully() should send recommission event to active RMNodes only

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.0-alpha1
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: graceful
• Hadoop Flags: Reviewed
• Assignee: Kuhu Shukla
• Reporter: Kuhu Shukla
• Created: 23/Nov/15 19:14
• Updated: 25/Oct/19 20:27
• Resolved: 22/Feb/16 15:31

Description

In refreshNodesGracefully(), during recommissioning, the entryset from getRMNodes() which has only active nodes (RUNNING, DECOMMISSIONING etc.) is used for checking 'decommissioned' nodes which are present in getInactiveRMNodes() map alone.

for (Entry<NodeId, RMNode> entry:rmContext.getRMNodes().entrySet()) { .........................
 // Recommissioning the nodes
        if (entry.getValue().getState() == NodeState.DECOMMISSIONING
            || entry.getValue().getState() == NodeState.DECOMMISSIONED) {
          this.rmContext.getDispatcher().getEventHandler()
              .handle(new RMNodeEvent(nodeId, RMNodeEventType.RECOMMISSION));
        }