You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Shawn Heisey (JIRA)" <ji...@apache.org> on 2015/07/23 20:57:04 UTC
[jira] [Commented] (SOLR-7825) Use slf4j consistently
[ https://issues.apache.org/jira/browse/SOLR-7825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14639362#comment-14639362 ]
Shawn Heisey commented on SOLR-7825:
------------------------------------
We should certainly fix problems like this.
All is not lost even without the fixes, though: If you're switching logging implementations, you should be adding the slf4j jar to intercept log4j calls and direct them through slf4j -- log4j-over-slf4j-X.Y.Z.jar -- and removing the actual log4j jar. Some of Solr's dependencies (zookeeper being a prime example) use log4j directly.
> Use slf4j consistently
> ----------------------
>
> Key: SOLR-7825
> URL: https://issues.apache.org/jira/browse/SOLR-7825
> Project: Solr
> Issue Type: Improvement
> Components: Server
> Affects Versions: 5.2.1
> Reporter: Oliver Schrenk
> Priority: Minor
>
> There are a few classes that directly rely on log4j to be on the classpath instead and don't use the slf4j logging facade. This creates problems when trying to switch the logging implementation.
> 1. org.apache.solr.core.ZkContainer
> https://github.com/apache/lucene-solr/blob/trunk/solr/core/src/java/org/apache/solr/core/ZkContainer.java#L218
> I don't know the impact of this change, but shouldn't this call `org.apache.solr.logging.MDCLoggingContext.clear()` ?
> 2. org.apache.solr.handler.component.RangeFacetProcessor and org.apache.solr.handler.component.RangeFacetRequest
> should use slf4j instead of log4j
> I had a stab at it at
> https://github.com/oschrenk/lucene-solr/commit/025b4802caf0360c63a3554af82e9ed4c94ff5a3#diff-7d822e8ff8ff21d88437652bbc894739R28
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org