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

Sangeetha Abdu Jyothi commented on YARN-5331:
---------------------------------------------

Thanks for the comments, [~subru]. 

1) I have addressed this in the latest patch.

2) getMaxPeriodicCapacity and getMinimumCapacityInInterval perform different 
functions. getMaxPeriodicCapacity finds the maximum at periodic offsets without 
considering the values in the interval between these time instants. On the 
other hand, getMinimumCapacityInInterval returns the minimum in the given 
interval. Hence I have renamed the function to getMaximumPeriodicCapacity.

3) Since this will lead to changes in the remaining JIRAs under YARN-5326, it 
might be helpful to make this a separate JIRA.  


> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-5331
>                 URL: https://issues.apache.org/jira/browse/YARN-5331
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Subru Krishnan
>            Assignee: Sangeetha Abdu Jyothi
>              Labels: oct16-medium
>         Attachments: YARN-5331.001.patch, YARN-5331.002.patch, 
> YARN-5331.003.patch, YARN-5331.004.patch, YARN-5331.005.patch, 
> YARN-5331.006.patch, YARN-5331.007.patch, YARN-5331.008.patch, 
> YARN-5331.009.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to