dlmarion commented on pull request #2467: URL: https://github.com/apache/accumulo/pull/2467#issuecomment-1031682906
@keith-turner reached out about my change in #2462 to highlight the fact that even after my change an IN_PROGRESS transaction may not be currently running as the status is set to IN_PROGRESS when the TransactionRunner deals with the first op in a fate transaction op chain. The subsequent op gets pushed back into ZK to be operated on at a later point. I believe that this change will still cancel a Fate Tx in this scenario as the tx will not be reserved so `Fate.cancel()` will set the status to failed in progress. -- 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]
