[ https://issues.apache.org/jira/browse/YARN-1365?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14006486#comment-14006486
]
Tsuyoshi OZAWA commented on YARN-1365:
--------------------------------------
[~adhoot], thank you for updating a patch. Looks good to me overall.
Minor nits:
We can removed following unused values:
{code}
// TestApplicationMasterLauncher.java
boolean thrown = false;
{code}
{code}
// TestRMRestart.java
Map<ApplicationId, ApplicationState> rmAppState =
rmState.getApplicationState();
{code}
> ApplicationMasterService to allow Register and Unregister of an app that was running
before restart
> ---------------------------------------------------------------------------------------------------
>
> Key: YARN-1365
> URL: https://issues.apache.org/jira/browse/YARN-1365
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: resourcemanager
> Reporter: Bikas Saha
> Assignee: Anubhav Dhoot
> Attachments: YARN-1365.001.patch, YARN-1365.002.patch, YARN-1365.003.patch, YARN-1365.initial.patch
>
>
> For an application that was running before restart, the ApplicationMasterService currently
throws an exception when the app tries to make the initial register or final unregister call.
These should succeed and the RMApp state machine should transition to completed like normal.
Unregistration should succeed for an app that the RM considers complete since the RM may have
died after saving completion in the store but before notifying the AM that the AM is free
to exit.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
|