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

Junping Du commented on YARN-1354:
----------------------------------

Thanks [~jlowe] for additional explanation which sounds good to me. About 
changes for writable credential, I think we already have a similar JIRA in 
YARN-668 which is talking about TokenIdentifier. One way is to wrapper it as PB 
object (keep writable fields as bytes), could be something like below:
{code}
required bytes credential;
optional new-field 1;  
optional new-field 2;
{code}
The other approach is to keep it as writable but do some extra work in 
readFields() that handle new field could be missing case. Any ideas on 
preference? However, this sounds a little away from this JIRA, may be YARN-668 
is a better place for discussion there?
The patch looks good in overall. Some trivial comments:
{code}
+  public abstract RecoveredApplicationsState loadApplicationsState()
+      throws IOException;
+
+  public abstract void storeApplication(ApplicationId appId,
+      ContainerManagerApplicationProto p) throws IOException;
+
+  public abstract void finishApplication(ApplicationId appId)
+      throws IOException;
+
+  public abstract void removeApplication(ApplicationId appId)
+      throws IOException;
+
{code}
Shall we change the name of finishApplication() to storeFinishedApplication() 
which sounds more precisely to actual work in store layer? (just like we use 
storeApplication() instead of startApplication()).

> Recover applications upon nodemanager restart
> ---------------------------------------------
>
>                 Key: YARN-1354
>                 URL: https://issues.apache.org/jira/browse/YARN-1354
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>    Affects Versions: 2.3.0
>            Reporter: Jason Lowe
>            Assignee: Jason Lowe
>         Attachments: YARN-1354-v1.patch, 
> YARN-1354-v2-and-YARN-1987-and-YARN-1362.patch, YARN-1354-v3.patch, 
> YARN-1354-v4.patch, YARN-1354-v5.patch
>
>
> The set of active applications in the nodemanager context need to be 
> recovered for work-preserving nodemanager restart



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

Reply via email to