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

Yang Wang commented on YARN-4166:
---------------------------------

Update the patch according to [~Naganarasimha]'s suggestion

# *updateContainerResource* has updated to be abstract, 
*DefaultContainerExecutor* has an empty implementation.
# *ResourceHandlerException* will be thrown when container resource update 
failed.
# *ContainerExecutor().updateContainerResource* failed, need to persist 
container resource change for recovery again.
# CGroupsCpuResourceHandlerImpl will not invoke *cGroupsHandler.deleteCGroup* 
when updateContainerResource, ResourceHandlerException will be caught in 
ContainerManagerImpl, then add to failedContainers
# *void updateContainerResource*
# Add test *testContainerManager.testUpdateContainerResourceFailed*
# fix checkstyle issue

> Support changing container cpu resource
> ---------------------------------------
>
>                 Key: YARN-4166
>                 URL: https://issues.apache.org/jira/browse/YARN-4166
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api, nodemanager, resourcemanager
>    Affects Versions: 2.8.0, 3.0.0-alpha2
>            Reporter: Jian He
>            Assignee: Yang Wang
>         Attachments: YARN-4166.001.patch, YARN-4166.002.patch, 
> YARN-4166.003.patch, YARN-4166.004.patch, YARN-4166-branch2.8-001.patch
>
>
> Memory resizing is now supported, we need to support the same for cpu.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to