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

Sandy Ryza commented on YARN-417:
---------------------------------

That's a good point.  I'm worried about taking it in this direction, though, 
for reasons of future compatibility.  If we decide to later include additional 
information with an AMResponse, staying consistent with this approach would 
require us to add additional methods to the callback handler, which would break 
API compatibility.  I don't have a clear answer on the best way to deal with 
this.  One option would be to make the callback handler an abstract class and 
not force users to implement methods like getClusterAvailableResources.  
Another would be to have a callback that returns the entire AMResponse.
                
> Add a poller that allows the AM to receive notifications when it is assigned 
> containers
> ---------------------------------------------------------------------------------------
>
>                 Key: YARN-417
>                 URL: https://issues.apache.org/jira/browse/YARN-417
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api, applications
>    Affects Versions: 2.0.3-alpha
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: AMRMClientAsync-1.java, AMRMClientAsync.java, 
> YARN-417-1.patch, YARN-417-2.patch, YARN-417-3.patch, YARN-417-4.patch, 
> YARN-417-4.patch, YARN-417.patch, YarnAppMaster.java, 
> YarnAppMasterListener.java
>
>
> Writing AMs would be easier for some if they did not have to handle 
> heartbeating to the RM on their own.

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