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 
> 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