[ 
https://issues.apache.org/jira/browse/SOLR-7825?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Oliver Schrenk updated SOLR-7825:
---------------------------------
    Attachment: slf4j.patch

> 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
>         Attachments: slf4j.patch
>
>
> 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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to