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

Jian He commented on YARN-2204:
-------------------------------

This logic was to test the container is indeed released. The patch replaced it 
to test scheduler attempt is removed.  we may have this back also?
{code}
    // the 2nd attempt released the 1st attempt's running container, when the
    // 2nd attempt finishes.
    Assert.assertFalse(schedulerNewAttempt.getLiveContainers().contains(
      containerId2));
{code}

> TestAMRestart#testAMRestartWithExistingContainers assumes CapacityScheduler
> ---------------------------------------------------------------------------
>
>                 Key: YARN-2204
>                 URL: https://issues.apache.org/jira/browse/YARN-2204
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: resourcemanager
>    Affects Versions: 2.5.0
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>            Priority: Trivial
>             Fix For: 2.5.0
>
>         Attachments: YARN-2204.patch, YARN-2204_addendum.patch
>
>
> TestAMRestart#testAMRestartWithExistingContainers assumes CapacityScheduler



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

Reply via email to