EdColeman commented on PR #3202: URL: https://github.com/apache/accumulo/pull/3202#issuecomment-1446538760
The use of sleepUninterruptibly will wait for the entire duration of the sleep, which seems to be a case that should be rare in our code to be the desired behavior. When that is truly intended we should make that explicit through the code and not as a side effect of the method. In general, it felt like we are using sleepUninterruptibly excessively to avoid handling or re-throwing InterruptedException. I was trying to move the code into a direction where we can improve InterruptException handling rather than assuming that the can be ignored. -- 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]
