EdColeman opened a new pull request, #3202:
URL: https://github.com/apache/accumulo/pull/3202
Replaces sleepUninterruptibly with static method in
a.a./core/util/UtilWaitThread that can be
interrupted, logs if an interrupt occurs and preserves the interrupt flag
for handing higher in the call stack
- replace `sleepUninterruptibly()` with `UtilWaitThread.sleep(long duration,
TimeUnit units)
- Improve interrupt handling:
- where InterruptException was already declared, throw an Interrupt
exception
- where exception handling in place (or in tests) throw an
IllegalStateException
- throw either AccumuloException or IOException when those were
previously declared.
- In places with busy loop, test the interrupt flag to break out of the
loop
- otherwise, maintain behaviour and ignore the interrupt.
- where the interrupt is not handled, tag with `// ignore interrupt
status` to allow for searching
- Deprecate UtilWaitThread.sleep(long duration) with the new method that
takes a TimeUnit
- Use TimeUnit.X with static imports for sleep, - changes some other
TimeUnit usages same file.
- Minor checkstyle fixes.
--
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]
