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

Sandy Ryza commented on YARN-521:
---------------------------------

I uploaded a new patch that adds in the checks mentioned by Bikas and 
Alejandro.  I moved the parameter documentation to constructors.  Regarding the 
priority comments, I think the new ones are clearer, but they're out of the 
scope this JIRA so I reverted them and I'll argue somewhere else if I want to 
make that change.  Regarding renaming allRacks to dedupedRacks, the code is 
doing the same thing as it was before for requests with locality relaxed, but I 
had to move some things around to accommodate disabling locality relaxation.
                
> Augment AM - RM client module to be able to request containers only at 
> specific locations
> -----------------------------------------------------------------------------------------
>
>                 Key: YARN-521
>                 URL: https://issues.apache.org/jira/browse/YARN-521
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api
>    Affects Versions: 2.0.3-alpha
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-521-1.patch, YARN-521.patch
>
>
> When YARN-392 and YARN-398 are completed, it would be good for AMRMClient to 
> offer an easy way to access their functionality

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