[
https://issues.apache.org/jira/browse/TIKA-2245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17698065#comment-17698065
]
Konstantin Gribov commented on TIKA-2245:
-----------------------------------------
[~lfcnassif], yeah {{commons-logging}} must be excluded when using
{{jcl-over-slf4j}}. In case of using Log4j2 bridge
{{org.apache.logging.log4j:log4j-jcl}} it's opposite (must be present in the
classpath) but still no need for explicit dependency, it would be brought
transitively.
{{jackcess}} should stay on {{commons-logging}} (at least without release with
breaking change) but on our side we should have exclusion and
{{jcl-over-slf4j}}.
Not sure if it should be {{tika-parser-*-module}} level though. I'd prefer it
in the {{tika-parsers-standard-package}} only.
That way if advanced downstream user choose one of the fine-grained
{{tika-parser-*-module}}s they add either {{jcl-over-slf4j}} or {{log4j-jcl}}
to their classpath. And in case of more mainstream usage
{{tika-parsers-standard-package}} brings convenient bridge without much hustle.
I updated [Logging wiki
page|https://cwiki.apache.org/confluence/display/TIKA/Logging] after 2.6.0 to
more or less represent current state of affairs. Maybe I should migrate it to
{{src/site}} in future. Confluence editor is so much pain in the arse when
adding/editing code blocks if you have more than one on a wiki page..
> Standardise logging
> -------------------
>
> Key: TIKA-2245
> URL: https://issues.apache.org/jira/browse/TIKA-2245
> Project: Tika
> Issue Type: Improvement
> Components: parser
> Affects Versions: 1.14, 1.15
> Reporter: Matthew Caruana Galizia
> Assignee: Konstantin Gribov
> Priority: Major
> Labels: logging
> Fix For: 1.15
>
>
> Tika parsers sometimes use Log4j's Logger, sometimes the JUL
> (java.util.logging) Logger and sometimes SLF4j.
> It would be better to standardise on a single facade, for the sake of not
> having to configure multiple loggers.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
