Hello, Carter and I have spent some time on overhauling the AsyncAppender background thread. The last state of the changes are available in the GitHub PR <https://github.com/apache/logging-log4j2/pull/452>. In a nutshell, the incorporated changes are as follows:
- AppenderControl catch clauses is demoted from Throwable to Exception. (Aligned with the pre-2.14.0 behaviour.) - AsyncAppender.AsyncThread is moved to AsyncAppenderEventForwarder with plenty of refactoring for simplification. - On shutdown, AsyncAppenderEventForwarder forwards all the remaining LogEvents, whereas AsyncAppender.AsyncThread was ignoring non- Log4LogEvents. - AsyncAppenderExceptionHandlingTest is improved for ThreadDeath. I would appreciate it if somebody can review the work. I want to get this in, fix any broken tests (if necessary), and make a new release. Kind regards.
