[ https://issues.apache.org/jira/browse/YARN-3985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14804746#comment-14804746 ]
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 (v6.3.4#6332)