kafka-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From guozh...@apache.org
Subject [kafka] branch 2.1 updated: KAFKA-7741: Streams exclude javax dependency (#6121)
Date Thu, 10 Jan 2019 23:51:48 GMT
This is an automated email from the ASF dual-hosted git repository.

guozhang pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new a20a26c  KAFKA-7741: Streams exclude javax dependency (#6121)
a20a26c is described below

commit a20a26ca657982cb65b5a360a1096dd1d460eba3
Author: John Roesler <vvcephei@users.noreply.github.com>
AuthorDate: Thu Jan 10 17:50:52 2019 -0600

    KAFKA-7741: Streams exclude javax dependency (#6121)
    
    As documented in https://issues.apache.org/jira/browse/KAFKA-7741,
    the javax dependency we receive transitively from connect is incompatible
    with SBT builds.
    
    Streams doesn't use the portion of Connect that needs the dependency,
    so we can fix the builds by simply excluding it.
    
    Reviewers: Bill Bejeck <bill@confluent.io>, Guozhang Wang <guozhang@confluent.io>
---
 build.gradle | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/build.gradle b/build.gradle
index 407ad4e..064bd2c 100644
--- a/build.gradle
+++ b/build.gradle
@@ -957,7 +957,13 @@ project(':streams') {
 
   dependencies {
     compile project(':clients')
-    compile project(':connect:json')  // this dependency should be removed after we unify
data API
+
+    // this dependency should be removed after we unify data API
+    compile(project(':connect:json')) {
+      // this transitive dependency is not used in Streams, and it breaks SBT builds
+      exclude module: 'javax.ws.rs-api'
+    }
+
     compile libs.slf4jApi
     compile libs.rocksDBJni
 


Mime
View raw message