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

Reply via email to