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

Anubhav Dhoot commented on YARN-3736:
-------------------------------------

Thanks [~subru] for the review.
Rebased the patch add addressed all feedback except the ZKRMstateStore
For deleting reservation, in the first step we remove the reservation and then 
we check if the plan state is empty because of this removal. If so we remove 
the plan state as well. Because of this two steps we cannot use a transaction 
for both. So we either use 2 different transactions or leave it as is. I have 
left it as is for now and let me know if you feel we still need to use 2 
SafeTransactions or do something else.




> Persist the Plan information, ie. accepted reservations to the RMStateStore 
> for failover
> ----------------------------------------------------------------------------------------
>
>                 Key: YARN-3736
>                 URL: https://issues.apache.org/jira/browse/YARN-3736
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: capacityscheduler, fairscheduler, resourcemanager
>            Reporter: Subru Krishnan
>            Assignee: Anubhav Dhoot
>         Attachments: YARN-3736.001.patch, YARN-3736.001.patch
>
>
> We need to persist the current state of the plan, i.e. the accepted 
> ReservationAllocations & corresponding RLESpareseResourceAllocations  to the 
> RMStateStore so that we can recover them on RM failover. This involves making 
> all the reservation system data structures protobuf friendly.



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

Reply via email to