Skip to content

Commit

Permalink
Fix AbstractMessageListenerContainer Javadoc regarding error log level
Browse files Browse the repository at this point in the history
This commit updates AbstractMessageListenerContainer's Javadoc
regarding the log level used in invokeErrorHandler() so that the
documentation aligns with the implementation, namely that errors will
logged at WARN level if no ErrorHandler has been registered.

Closes spring-projectsgh-30730
  • Loading branch information
sbrannen authored and mdeinum committed Jun 29, 2023
1 parent 3552cc1 commit 3e3e476
Showing 1 changed file with 4 additions and 3 deletions.
Expand Up @@ -46,7 +46,7 @@
*
* <p><b>NOTE:</b> The default behavior of this message listener container is to
* <b>never</b> propagate an exception thrown by a message listener up to the JMS
* provider. Instead, it will log any such exception at the error level.
* provider. Instead, it will log any such exception at {@code WARN} level.
* This means that from the perspective of the attendant JMS provider no such
* listener will ever fail. However, if error handling is necessary, then
* an implementation of the {@link ErrorHandler} strategy may be provided to
Expand Down Expand Up @@ -877,7 +877,7 @@ else if (isSubscriptionDurable()) {

/**
* Handle the given exception that arose during listener execution.
* <p>The default implementation logs the exception at warn level,
* <p>The default implementation logs the exception at {@code WARN} level,
* not propagating it to the JMS provider &mdash; assuming that all handling of
* acknowledgement and/or transactions is done by this listener container.
* This can be overridden in subclasses.
Expand Down Expand Up @@ -916,7 +916,8 @@ protected void invokeExceptionListener(JMSException ex) {
}

/**
* Invoke the registered ErrorHandler, if any. Log at warn level otherwise.
* Invoke the registered {@link #getErrorHandler() ErrorHandler} if any.
* Log at {@code WARN} level otherwise.
* @param ex the uncaught error that arose during JMS processing
* @see #setErrorHandler
*/
Expand Down

0 comments on commit 3e3e476

Please sign in to comment.