You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Shannon Carey (JIRA)" <ji...@apache.org> on 2017/04/04 18:03:41 UTC

[jira] [Commented] (ZOOKEEPER-1371) Remove dependency on log4j in the source code.

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

Shannon Carey commented on ZOOKEEPER-1371:
------------------------------------------

I second the suggestion by [~sslavic] to split out the server distro from other stuff, such as the jar that clients would use. Otherwise everyone who uses the ZK client has to add an exclusion for ZK's log4j dependency in order to make sure their slf4j works right (if they're not using Log4j).

> Remove dependency on log4j in the source code.
> ----------------------------------------------
>
>                 Key: ZOOKEEPER-1371
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1371
>             Project: ZooKeeper
>          Issue Type: Bug
>    Affects Versions: 3.4.0, 3.4.1, 3.4.2, 3.4.3
>            Reporter: Mahadev konar
>            Assignee: Mohammad Arshad
>              Labels: patch
>             Fix For: 3.5.2, 3.6.0
>
>         Attachments: ZOOKEEPER-1371-05.patch, ZOOKEEPER-1371.patch, ZOOKEEPER-1371.patch, ZOOKEEPER-1371.patch, ZOOKEEPER-1371.patch
>
>
> ZOOKEEPER-850 added slf4j to ZK. We still depend on log4j in our codebase. We should remove the dependency on log4j so that we can make logging pluggable.



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