From 7b8bb383224bfacd1aad0eef43666063e6644b00 Mon Sep 17 00:00:00 2001 From: James Carr Date: Mon, 14 Nov 2011 22:08:39 -0600 Subject: [PATCH] AMQP-200 changes log messages from debug to warn added @author tag as per contribution guidelines changed isDebugEnabled and isInfoEnabled to isWarnEnabled. removed else if that never gets called now. Squashed --- .../rabbit/listener/AbstractMessageListenerContainer.java | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/AbstractMessageListenerContainer.java b/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/AbstractMessageListenerContainer.java index 0e2d90d9..a7253442 100644 --- a/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/AbstractMessageListenerContainer.java +++ b/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/AbstractMessageListenerContainer.java @@ -34,6 +34,7 @@ import com.rabbitmq.client.Channel; * @author Mark Pollack * @author Mark Fisher * @author Dave Syer + * @author James Carr */ public abstract class AbstractMessageListenerContainer extends RabbitAccessor implements BeanNameAware, DisposableBean, SmartLifecycle { @@ -422,11 +423,8 @@ public abstract class AbstractMessageListenerContainer extends RabbitAccessor im protected void invokeErrorHandler(Throwable ex) { if (this.errorHandler != null) { this.errorHandler.handleError(ex); - } else if (logger.isDebugEnabled()) { - logger.debug("Execution of Rabbit message listener failed, and no ErrorHandler has been set.", ex); - } else if (logger.isInfoEnabled()) { - logger.info("Execution of Rabbit message listener failed, and no ErrorHandler has been set: " - + ex.getClass() + ": " + ex.getMessage()); + } else if (logger.isWarnEnabled()) { + logger.warn("Execution of Rabbit message listener failed, and no ErrorHandler has been set.", ex); } }