dlmarion commented on pull request #2329:
URL: https://github.com/apache/accumulo/pull/2329#issuecomment-967567259


   @keith-turner - can you take a look at my changes in 1755864 to see if this 
implementation seems sane. The high level notes are that I modified ZooStore to:
   
     1. Keep a reference to the thread that is reserving the tx
     2. Added a cancel method to call interrupt on the thread
     
   The calling Thread in most cases should be the Fate.TransactionRunner 
threads which already handle the Exception and transition the tx to 
FAILED_IN_PROGRESS and then ultimately unreserves the tx. I also fixed up a 
test and added FateIT with a test for preemption. Finally I took a stab at 
determining which FaTE transaction types should preempt.


-- 
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]


Reply via email to