keith-turner commented on PR #5016: URL: https://github.com/apache/accumulo/pull/5016#issuecomment-2448671797
> Regarding consistency - what's the behavior in earlier releases? I think they both should be consistent with that answer. Looking at the 3.1 code it would delete the fate tx no matter what state it was in. In 3.1 there as no check in the fate store of the status. I would recommend adding a forceDelete() call to the fate store that will delete no matter what the status. The admin code could call this and would have the same behavior as 3.1. However the non-admin fate code that calls delete would have this stronger check of the status that prevents bugs. Do not expect the non admin fate code to delete in progress transactions. -- 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]
