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

Karthik Kambatla commented on YARN-1959:
----------------------------------------

Thanks for the clarification here and offline. I understand why the headroom 
needs to policy specific. Couple of nits:
# In FifoPolicy and FairSharePolicy, we can avoid one instance of Resource - 
{{queueAvailable}}, and use an int for memory instead. May be, we should just 
use two ints in DRFPolicy as well. 
# TestFSAppAttempt#VerifyHeadroom should be verifyHeadroom. 



> Fix headroom calculation in Fair Scheduler
> ------------------------------------------
>
>                 Key: YARN-1959
>                 URL: https://issues.apache.org/jira/browse/YARN-1959
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Sandy Ryza
>            Assignee: Anubhav Dhoot
>         Attachments: YARN-1959.001.patch, YARN-1959.prelim.patch
>
>
> The Fair Scheduler currently always sets the headroom to 0.



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

Reply via email to