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

Junping Du commented on YARN-2209:
----------------------------------

Thanks [~jianhe] for the patch and [~rohithsharma] for review! I think this is 
a reasonable change and patch itself looks good to me. However, I have concern 
that it could break existing YARN applications that run with old version 
ApplicationMasterProtocol which looks forward to a RESYNC command rather than 
an exception in response. More discussions with broadly people in community are 
needed, I think.

> Replace AM resync/shutdown command with corresponding exceptions
> ----------------------------------------------------------------
>
>                 Key: YARN-2209
>                 URL: https://issues.apache.org/jira/browse/YARN-2209
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-2209.1.patch, YARN-2209.2.patch, YARN-2209.3.patch, 
> YARN-2209.4.patch, YARN-2209.5.patch
>
>
> YARN-1365 introduced an ApplicationMasterNotRegisteredException to indicate 
> application to re-register on RM restart. we should do the same for 
> AMS#allocate call also.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to