[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-2342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15955532#comment-15955532
 ] 

Shannon Carey commented on ZOOKEEPER-2342:
------------------------------------------

I second the advice of [~anomalizer], "Do not bundle any binding as part of 
non-server artifacts a.k.a. libraries. This is a place where the power/choice 
should solely lie in the hands of the person integrating the libraries into 
their applications."

Otherwise people who use a Zookeeper client in their code (and don't use log4j) 
have to add exclusions for log4j & the slf4j-log4j12 binding in order to avoid 
the "SLF4J: Detected both log4j-over-slf4j.jar AND slf4j-log4j12.jar on the 
class path, preempting StackOverflowError." error caused by multiple bindings 
in their app. Well-behaved sl4j based libraries should only have a test-scope 
or {{<optional>true</optional>}} dependency on slf4j bindings & logger 
implementations.

> Migrate to Log4J 2.
> -------------------
>
>                 Key: ZOOKEEPER-2342
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2342
>             Project: ZooKeeper
>          Issue Type: Bug
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>             Fix For: 3.6.0
>
>         Attachments: ZOOKEEPER-2342.001.patch
>
>
> ZOOKEEPER-1371 removed our source code dependency on Log4J.  It appears that 
> this also removed the Log4J SLF4J binding jar from the runtime classpath.  
> Without any SLF4J binding jar available on the runtime classpath, it is 
> impossible to write logs.
> This JIRA investigated migration to Log4J 2 as a possible path towards 
> resolving the bug introduced by ZOOKEEPER-1371.  At this point, we know this 
> is not feasible short-term.  This JIRA remains open to track long-term 
> migration to Log4J 2.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to