You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Christian Schneider (JIRA)" <ji...@apache.org> on 2011/09/05 15:44:16 UTC
[jira] [Resolved] (CAMEL-4414) CamelLogger should not be a
Processor and a log class at the same time and it should not have
ServiceSupport
[ https://issues.apache.org/jira/browse/CAMEL-4414?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Schneider resolved CAMEL-4414.
----------------------------------------
Resolution: Fixed
> CamelLogger should not be a Processor and a log class at the same time and it should not have ServiceSupport
> ------------------------------------------------------------------------------------------------------------
>
> Key: CAMEL-4414
> URL: https://issues.apache.org/jira/browse/CAMEL-4414
> Project: Camel
> Issue Type: Improvement
> Components: camel-core
> Affects Versions: 2.8.0
> Reporter: Christian Schneider
> Assignee: Christian Schneider
> Fix For: 2.9.0
>
>
> CamelLogger currently does too many things.
> 1. It is a log class that stores a log level and a slf4j logger. It suppports some methods for logging with the set level
> 2. It is a processor for logging exchanges
> 3. It extends ServiceSupport even though it has no services (is apparently needed fro ThroughputLogger
> So what I propose is:
> - Leave the current class as is but mark as @Deprecated
> - Introduce a new CamelLogger that only supports 1.
> - Introduce a CamelLogProcessor in processor that only supports 2.
> - Make ThroughputLogger not extend CamelLogger but instead delegate to the new CamelLogger
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira