[ 
https://issues.apache.org/jira/browse/YARN-791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13684887#comment-13684887
 ] 

Hitesh Shah commented on YARN-791:
----------------------------------

[~sandyr] I put in a similar comment on YARN-727. In my opinion, we could make 
a cleaner api where passing null as the state filter returns all nodes ( both 
in the RPC as well as the REST webservices ). Adding a state acts as a filter. 
For usage in the CLI tools that returned only healthy nodes, the tools could be 
changed to pass in the correct state filter to the api. 

If there are other components that rely on this, we could change them to start 
passing in the correct state to filter for healthy nodes only. 

Obviously this is a behavior change which may be cause problems for a lot of 
reasons. Given that the CLI tools and MapReduce are probably the only 2 major 
users of this api, it might be better to bite the bullet and make the changes 
now? 

Comments?
                
> Ensure that RM RPC APIs that return nodes are consistent with /nodes REST API
> -----------------------------------------------------------------------------
>
>                 Key: YARN-791
>                 URL: https://issues.apache.org/jira/browse/YARN-791
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api, resourcemanager
>    Affects Versions: 2.0.4-alpha
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-791-1.patch, YARN-791.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to