kevinrr888 commented on PR #4524: URL: https://github.com/apache/accumulo/pull/4524#issuecomment-2145316505
I have made the following changes in https://github.com/apache/accumulo/pull/4524/commits/e6671fbb903da53014bd64e53ad125351add6a04: - Combined the LockID and reservation attempt UUID into one new object: FateReservation - Now cover edge case with tryReserve() where a write may make it to the server, but the server dies before a response is received - New FATE Thread: DeadReservationCleaner which deletes reservations held by Managers that have since died - Created new classes: ColumnValueMappingIterator and ReservationMappingIterator. ColumnValueMappingIterator abstracts out the common functionality of ReservationMappingIterator and StatusMappingIterator. ReservationMappingIterator is an iterator used for determining if the reservation column for a FateId has a FateReservation set or not - Expanded/improved/simplified MultipleStoresIT tests There are also some comments/questions labeled "TODO 4131" that I would like input on. I also have to resolve the merge conflicts which I will do now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
