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

Karthik Kambatla commented on YARN-2080:
----------------------------------------

Thanks for the patch, Subru. Just skimmed through it, the patch understandably 
didn't apply on trunk. Couple of high-level comments:
# Can we make the ReservationSystem a service and make it a part of 
RMActiveServices. 
# Looks like we ll need a different ReservationSystem implementation per policy 
(or scheduler). Can we put as much functionality as possible in 
AbstractReservationSystem and have CapacityReservationSystem extend it? This 
would be similar to how the scheduler hierarchy works today.

> Admission Control: Integrate Reservation subsystem with ResourceManager
> -----------------------------------------------------------------------
>
>                 Key: YARN-2080
>                 URL: https://issues.apache.org/jira/browse/YARN-2080
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Subramaniam Venkatraman Krishnan
>            Assignee: Subramaniam Venkatraman Krishnan
>         Attachments: YARN-2080.patch
>
>
> This JIRA tracks the integration of Reservation subsystem data structures 
> introduced in YARN-1709 with the YARN RM. This is essentially end2end wiring 
> of YARN-1051.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to