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

Vinod Kumar Vavilapalli commented on YARN-2260:
-----------------------------------------------

As I see it, if we don't have this patch, the only downside is that schedulers 
will get duplicate events about container, the container-state-machine will 
throw invalid-transition, but the RM doesn't crash.

Anyways, the patch looks good, +1. Checking this in..

> Add containers to launchedContainers list in RMNode on container recovery
> -------------------------------------------------------------------------
>
>                 Key: YARN-2260
>                 URL: https://issues.apache.org/jira/browse/YARN-2260
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-2260.1.patch, YARN-2260.2.patch
>
>
> The justLaunchedContainers map in RMNode should be re-populated when 
> container is sent from NM for recovery.



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

Reply via email to