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

Subru Krishnan commented on YARN-4358:
--------------------------------------

[~asuresh], you need not update the Javadoc of _getReservationById_. The problem is caused because we are specifying *Set* inside _{@ link}_ so the fix should be just be to update the Javadoc of the return parameter of _getReservations_ to:
bq @return set of active {@link ReservationAllocation}s for the specified user at the requested time

> Improve relationship between SharingPolicy and ReservationAgent
> ---------------------------------------------------------------
>
>                 Key: YARN-4358
>                 URL: https://issues.apache.org/jira/browse/YARN-4358
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: capacityscheduler, fairscheduler, resourcemanager
>            Reporter: Carlo Curino
>            Assignee: Carlo Curino
>             Fix For: 2.8.0
>
>         Attachments: YARN-4358.2.patch, YARN-4358.3.patch, YARN-4358.4.patch, YARN-4358.addendum.patch, YARN-4358.patch
>
>
> At the moment an agent places based on available resources, but has no visibility to extra constraints imposed by the SharingPolicy. While not all constraints are easily represented some (e.g., max-instantaneous resources) are easily represented.



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