[CAMEL-964] loggingErrorHandler should default to ERROR and LoggingErrorHandlerBuilder should operate as a builder

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.0
• Fix Version/s: 1.5.0
• Component/s: camel-core
• Patch Info: Patch Available
• Assignee: Claus Ibsen
• Reporter: Martin Gilday
• Created: 07/Oct/08 12:27
• Updated: 16/Feb/09 05:51
• Resolved: 09/Oct/08 11:44

Description

Please see attached patch which changes the default level created by the LoggingErrorHandlerBuilder to ERROR instead of INFO.

Also some of the methods on this builder to not return "this". This breaks the chain when attempting to use the builder resulting in:

LoggingErrorHandlerBuilder errorBuilder = loggingErrorHandler("my.logger.name");
errorBuilder.setLevel(LoggingLevel.ERROR);
errorHandler(errorBuilder);

instead of:

errorHandler(loggingErrorHandler("mylogger.name").level(LoggingLevel.ERROR));

I have added on builder methods for "level" and for "log". I have left the properties that currently exist as following the pattern on DeadLetterChannelBuilder the builder method names should not include "set". Whoever applies the patch may wish to remove the getters and setters, depending how they feel this will affect existing users.