kafka git commit: KAFKA-4018; Streams causing older slf4j-log4j library to be packaged along with newer version

[ij...@apache.org]

Repository: kafka
Updated Branches:
  refs/heads/trunk 27fc11d1c -> 2e3722a23


KAFKA-4018; Streams causing older slf4j-log4j library to be packaged along with newer version

This is a regression caused by 0bb1d3ae.

After that commit, Streams no longer has a direct dependency on slf4j-log4j12, but zkclient
has a dependency on an older version of slf4j-log4j12, so we get a transitive dependency on
the older version.

The fix is to simply exclude the undesired dependencies from the zkclient dependency.

Author: Ismael Juma <is...@juma.me.uk>

Reviewers: Guozhang Wang <wa...@gmail.com>

Closes #1704 from ijuma/kafka-4018-streams-duplicate-slf4j-log4j


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/2e3722a2
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/2e3722a2
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/2e3722a2

Branch: refs/heads/trunk
Commit: 2e3722a234df5c0cd2e91c258cc0e4d825ee0626
Parents: 27fc11d
Author: Ismael Juma <is...@juma.me.uk>
Authored: Thu Aug 4 02:03:57 2016 +0100
Committer: Ismael Juma <is...@juma.me.uk>
Committed: Thu Aug 4 02:03:57 2016 +0100

----------------------------------------------------------------------
 build.gradle | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/2e3722a2/build.gradle
----------------------------------------------------------------------
diff --git a/build.gradle b/build.gradle
index ead2520..5889f71 100644
--- a/build.gradle
+++ b/build.gradle
@@ -681,6 +681,9 @@ project(':streams') {
     compile libs.rocksDBJni
     // this dependency should be removed after KIP-4
     compile (libs.zkclient) {
+      // users should be able to choose the logging implementation (and slf4j bridge)
+      exclude module: 'slf4j-log4j12'
+      exclude module: 'log4j'
       exclude module: 'jline'
       exclude module: 'netty'
     }