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

Jian He commented on YARN-1644:
-------------------------------

thanks [~mding]! one moment on the patch:

- NodeStatusUpdaterImpl#getIncreasedContainers and the change in 
ContainerManagerImpl:
If RM restarts right after NM sends the increased containers, NM may not yet 
update the container status and still sends the old resource and therefore RM 
will recover the container with old resource. How about we re-use the same 
containerStatus object for the increasedContainer ? this way we can also avoid 
this situation.

> RM-NM protocol changes and NodeStatusUpdater implementation to support 
> container resizing
> -----------------------------------------------------------------------------------------
>
>                 Key: YARN-1644
>                 URL: https://issues.apache.org/jira/browse/YARN-1644
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>            Reporter: Wangda Tan
>            Assignee: MENG DING
>         Attachments: YARN-1644-YARN-1197.4.patch, 
> YARN-1644-YARN-1197.5.patch, YARN-1644.1.patch, YARN-1644.2.patch, 
> YARN-1644.3.patch, yarn-1644.1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to