cnauroth opened a new pull request, #5060:
URL: https://github.com/apache/hadoop/pull/5060

   ### Description of PR
   
   YARN cluster metrics expose counts of NodeManagers in various states 
including active and decommissioned. However, these metrics don't expose 
NodeManagers that are currently in the process of decommissioning. This can 
look a little spooky to a consumer of these metrics. First, the node drops out 
of the active count, so it seems like a node just vanished. Then, later 
(possibly hours later with consideration of graceful decommission), it comes 
back into existence in the decommissioned count.
   
   This issue tracks adding the decommissioning count to the metrics 
ResourceManager RPC. This also enables exposing it in the `yarn top` output. 
This metric is already visible through the REST API, so there isn't any change 
required there.
   
   ### How was this patch tested?
   
   The patch adds new unit tests for the ResourceManager RPC, correct merging 
of the metric through the router service and `yarn top`.
   
   ### For code changes:
   
   - [X] Does the title or this PR starts with the corresponding JIRA issue id 
(e.g. 'HADOOP-17799. Your PR title ...')?
   - [X] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   - [X] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [X] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to