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

Tao Yang edited comment on YARN-8774 at 9/22/18 12:17 AM:
----------------------------------------------------------

Thanks [~eepayne] for the review.
Attached v2 patch to improve the UT with adding check for killed state.
There are somewhere different in branch-2.8 (it also exist in branch-2.8) and 
branch-2, especially in branch2.8.
Backported this patch to these two branches and attached them also for review.


was (Author: tao yang):
Thanks [~eepayne] for the review.
Attached v2 patch to improve the UT with adding check for killed state.
There are somewhat different in branch-2.8 (it also exist in branch-2.8) and 
branch-2, especially in branch2.8.
Backported this patch to these two branches and attached them also for review.

> Memory leak when CapacityScheduler allocates from reserved container with 
> non-default label
> -------------------------------------------------------------------------------------------
>
>                 Key: YARN-8774
>                 URL: https://issues.apache.org/jira/browse/YARN-8774
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: capacityscheduler
>    Affects Versions: 3.2.0, 2.8.5
>            Reporter: Tao Yang
>            Assignee: Tao Yang
>            Priority: Critical
>         Attachments: YARN-8774.001.patch, YARN-8774.002.patch, 
> YARN-8774.branch-2.001.patch, YARN-8774.branch-2.8.001.patch
>
>
> The cause is that the RMContainerImpl instance of reserved container lost its 
> node label expression, when scheduler reserves containers for non-default 
> node-label requests, it will be wrongly added into 
> LeafQueue#ignorePartitionExclusivityRMContainers and never be removed.
> To reproduce this memory leak:
> (1) create reserved container
> RegularContainerAllocator#doAllocation:  create RMContainerImpl instanceA 
> (nodeLabelExpression="")
> LeafQueue#allocateResource:  RMContainerImpl instanceA is put into  
> LeafQueue#ignorePartitionExclusivityRMContainers
> (2) allocate from reserved container
> RegularContainerAllocator#doAllocation: create RMContainerImpl instanceB 
> (nodeLabelExpression="test-label")
> (3) From now on, RMContainerImpl instanceA will be left in memory (be kept in 
> LeafQueue#ignorePartitionExclusivityRMContainers) forever until RM restarted



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
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