[GitHub] [accumulo] keith-turner opened a new pull request, #3747: adds validation to prevent possible deadlock

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner opened a new pull request, #3747:
URL: https://github.com/apache/accumulo/pull/3747

   (no comment)


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] keith-turner commented on pull request #3747: adds validation to prevent possible deadlock

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on PR #3747:
URL: https://github.com/apache/accumulo/pull/3747#issuecomment-1714756924

   > It doesn't look like this can happen. Is this just being added to help us catch some future mistake?
   
   Yes it would be very easy to make a code change in a function that calls this function that breaks the expectation.  Adding the validation may catch that in test before it has chance to cause deadlock somewhere, acquiring the locks out of order does not always cause deadlock.


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] keith-turner merged pull request #3747: adds validation to prevent possible deadlock

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner merged PR #3747:
URL: https://github.com/apache/accumulo/pull/3747


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org