[ https://issues.apache.org/jira/browse/YARN-791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13685023#comment-13685023 ]
Sandy Ryza commented on YARN-791: --------------------------------- [~vinodkv], my concern about a single static factory in GetClusterNodesRequest was that giving a node and setting all states to true would make no sense. The way I was envisioning things, the defaults for the fields would be null/false, which would mean that we return the NEW/RUNNING nodes that the REST API returns by default. Would there be issues with making both the REST API and this API return only RUNNING nodes by default (no NEW)? In that case, I think Hitesh's proposal would make a lot of sense, and we could set RUNNING as the default in the proto file. > 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