kevinrr888 opened a new pull request, #5651: URL: https://github.com/apache/accumulo/pull/5651
Resolves a deadlock in Tablet related code. Expected lock order is refreshLock -> logLock -> tablet, but `Tablet.completeClose()` would lock tablet -> refreshLock -> logLock, which could (and did) result in deadlocks. ChaoticBalancerIT would previously timeout almost every run. I ran it 10+ times with these changes with no deadlocks. I did not find any other situations where we do not acquire the locks in the order refreshLock -> logLock -> tablet, so Tablet code should now be deadlock free. closes #5597 -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
