LOG4J2-1160 AsyncLoggerDisruptor should not log status message at FATAL level when logging is attempted after the Disruptor was shut down
Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/6207ea6b Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/6207ea6b Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/6207ea6b Branch: refs/heads/LOG4J2-1278-gc-free-logger Commit: 6207ea6b8545e89ca82e39b09d3552b8d638e0f4 Parents: f1e5063 Author: rpopma <[email protected]> Authored: Wed Mar 9 08:22:36 2016 +1100 Committer: rpopma <[email protected]> Committed: Wed Mar 9 08:22:36 2016 +1100 ---------------------------------------------------------------------- .../apache/logging/log4j/core/async/AsyncLoggerDisruptor.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/6207ea6b/log4j-core/src/main/java/org/apache/logging/log4j/core/async/AsyncLoggerDisruptor.java ---------------------------------------------------------------------- diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/async/AsyncLoggerDisruptor.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/async/AsyncLoggerDisruptor.java index 1188b73..c55e1c9 100644 --- a/log4j-core/src/main/java/org/apache/logging/log4j/core/async/AsyncLoggerDisruptor.java +++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/async/AsyncLoggerDisruptor.java @@ -179,7 +179,7 @@ class AsyncLoggerDisruptor { */ private boolean hasLog4jBeenShutDown(final Disruptor<RingBufferLogEvent> aDisruptor) { if (aDisruptor == null) { // LOG4J2-639 - LOGGER.fatal("Ignoring log event after log4j was shut down"); + LOGGER.error("Ignoring log event after log4j was shut down"); return true; } return false; @@ -193,7 +193,7 @@ class AsyncLoggerDisruptor { // was shut down, which could cause the publishEvent method to hang and never return. disruptor.publishEvent(translator); } catch (final NullPointerException npe) { - LOGGER.fatal("[{}] Ignoring log event after log4j was shut down.", contextName); + LOGGER.error("[{}] Ignoring log event after log4j was shut down.", contextName); } }
