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

Karthik Kambatla commented on YARN-2083:
----------------------------------------

Firstly, thanks for working on this long-standing issue. 

Correct me if I am wrong. Ideally, the check should be {{currentUsage + 
newContainerCapacity <= maxShare}}. Changing it from {{currentUsage <= 
maxShare}} to {{currentUsage < maxShare}} could address some cases, but not all 
the cases. Can we look at fixing it for all cases, moving the place 
assignContainerPreCheck is called if need be. 

> In fair scheduler, Queue should not been assigned more containers when its 
> usedResource had reach the maxResource limit
> -----------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-2083
>                 URL: https://issues.apache.org/jira/browse/YARN-2083
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: fairscheduler
>    Affects Versions: 2.3.0
>            Reporter: Yi Tian
>              Labels: BB2015-05-TBR, oct16-medium
>         Attachments: YARN-2083-1.patch, YARN-2083-2.patch, YARN-2083-3.patch, 
> YARN-2083-4.patch, YARN-2083.patch
>
>
> In fair scheduler, FSParentQueue and FSLeafQueue do an 
> assignContainerPreCheck to guaranty this queue is not over its limit.
> But the fitsIn function in Resource.java did not return false when the 
> usedResource equals the maxResource.
> I think we should create a new Function "fitsInWithoutEqual" instead of 
> "fitsIn" in this case.



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

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