Subru Krishnan commented on YARN-3985:

Thanks [~adhoot] for the patch. LGTM, just a minor nit - 
*TestSchedulerPlanFollowerBase::testPlanFollowerRecovery()* is not used so can 
be removed.

Kindly open a JIRA for removing _updateReservation_ from the State store as 
it's redundant.

> Make ReservationSystem persist state using RMStateStore reservation APIs 
> -------------------------------------------------------------------------
>                 Key: YARN-3985
>                 URL: https://issues.apache.org/jira/browse/YARN-3985
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Anubhav Dhoot
>            Assignee: Anubhav Dhoot
>         Attachments: YARN-3985.001.patch, YARN-3985.002.patch, 
> YARN-3985.002.patch, YARN-3985.002.patch
> YARN-3736 adds the RMStateStore apis to store and load reservation state. 
> This jira adds the actual storing of state from ReservationSystem.

This message was sent by Atlassian JIRA

Reply via email to