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

Reply via email to