[
https://issues.apache.org/jira/browse/YARN-594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13636923#comment-13636923
]
Bikas Saha commented on YARN-594:
---------------------------------
Why do we need to change rm1.stop()?
{code}
- rm1.stop();
// start new RM
MockRM rm2 = new MockRM(conf, memStore);
@@ -378,7 +377,12 @@ public void testRMRestartOnMaxAppAttempts() throws
Exception {
Assert.assertNull(rm2.getRMContext().getRMApps()
.get(app1.getApplicationId()));
- // stop the RM
+ // verify that app2 is stored, app1 is removed
+ Assert.assertNotNull(rmAppState.get(app2.getApplicationId()));
+ Assert.assertNull(rmAppState.get(app1.getApplicationId()));
+
+ // stop the RM
+ rm1.stop();
{code}
> Update test and add comments in YARN-534
> ----------------------------------------
>
> Key: YARN-594
> URL: https://issues.apache.org/jira/browse/YARN-594
> Project: Hadoop YARN
> Issue Type: Bug
> Reporter: Jian He
> Assignee: Jian He
> Attachments: YARN-594.1.patch
>
>
> This jira is simply to add some comments in the patch YARN-534 and update the
> test case
--
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
