In addition to swallowing the interrupt, the class also swallows the log event. It does write a notification into the status manager though. I would like to see the log event logged. I think that the Logger methods should be noncancelable. This would also be consistent with their signature: they do not indicate that they are cancelable by a throwing InterruptedException clause. If there were enough interest, the exact behavior may be configurable. One option could be that AbstactSocketAppender do not wait for free queue space at all if the thread is interrupted. |