Thanks for the patch Anubhav Dhoot and the review Subru Krishnan
I guess it looks pretty straightforward. Minor nit though :
- TestReservationSystemWithRMHA.testSubmitReservationAndCheckAfterFailover() uses explicit Thread.sleep to wait for the plan followers to synchronize. Is it maybe better to explicitly call ReservationSystem.synchronizePlan with the plan name and not have to perform a thread sleep and reduce possible test flakiness.
+1 pending your decision on the above.