[
https://issues.apache.org/jira/browse/KAFKA-7031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16510631#comment-16510631
]
ASF GitHub Bot commented on KAFKA-7031:
---------------------------------------
ewencp closed pull request #5190: KAFKA-7031 : Connect API shouldn't depend on
jersey
URL: https://github.com/apache/kafka/pull/5190
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/build.gradle b/build.gradle
index b796c218e21..7db8ddcf6d1 100644
--- a/build.gradle
+++ b/build.gradle
@@ -1210,7 +1210,7 @@ project(':connect:api') {
dependencies {
compile project(':clients')
compile libs.slf4jApi
- compile libs.jerseyContainerServlet
+ compile libs.jaxrsApi
testCompile libs.junit
@@ -1450,6 +1450,7 @@ project(':connect:basic-auth-extension') {
testCompile project(':clients').sourceSets.test.output
testRuntime libs.slf4jlog4j
+ testRuntime libs.jerseyContainerServlet
}
javadoc {
diff --git a/gradle/dependencies.gradle b/gradle/dependencies.gradle
index ce2db721025..c9f6f0349a5 100644
--- a/gradle/dependencies.gradle
+++ b/gradle/dependencies.gradle
@@ -60,6 +60,7 @@ versions += [
log4j: "1.2.17",
scalaLogging: "3.9.0",
jaxb: "2.3.0",
+ jaxrs: "2.1",
jfreechart: "1.0.0",
jopt: "5.0.4",
junit: "4.12",
@@ -101,6 +102,7 @@ libs += [
jacksonDatabind:
"com.fasterxml.jackson.core:jackson-databind:$versions.jackson",
jacksonJaxrsJsonProvider:
"com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:$versions.jackson",
jaxbApi: "javax.xml.bind:jaxb-api:$versions.jaxb",
+ jaxrsApi: "javax.ws.rs:javax.ws.rs-api:$versions.jaxrs",
jettyServer: "org.eclipse.jetty:jetty-server:$versions.jetty",
jettyClient: "org.eclipse.jetty:jetty-client:$versions.jetty",
jettyServlet: "org.eclipse.jetty:jetty-servlet:$versions.jetty",
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Kafka Connect API module depends on Jersey
> ------------------------------------------
>
> Key: KAFKA-7031
> URL: https://issues.apache.org/jira/browse/KAFKA-7031
> Project: Kafka
> Issue Type: Bug
> Components: KafkaConnect
> Affects Versions: 2.0.0
> Reporter: Randall Hauch
> Assignee: Magesh kumar Nandakumar
> Priority: Blocker
> Fix For: 2.0.0, 2.1.0
>
>
> The Kafka Connect API module for 2.0.0 brings in Jersey dependencies. When I
> run {{mvn dependency:tree}} on a project that depends only on the snapshot
> version of {{org.apache.kafka:kafka-connect-api}}, the following are shown:
> {noformat}
> [INFO] +- org.apache.kafka:connect-api:jar:2.0.0-SNAPSHOT:compile
> [INFO] | +- org.slf4j:slf4j-api:jar:1.7.25:compile
> [INFO] | \-
> org.glassfish.jersey.containers:jersey-container-servlet:jar:2.27:compile
> [INFO] | +-
> org.glassfish.jersey.containers:jersey-container-servlet-core:jar:2.27:compile
> [INFO] | | \-
> org.glassfish.hk2.external:javax.inject:jar:2.5.0-b42:compile
> [INFO] | +- org.glassfish.jersey.core:jersey-common:jar:2.27:compile
> [INFO] | | +- javax.annotation:javax.annotation-api:jar:1.2:compile
> [INFO] | | \- org.glassfish.hk2:osgi-resource-locator:jar:1.0.1:compile
> [INFO] | +- org.glassfish.jersey.core:jersey-server:jar:2.27:compile
> [INFO] | | +- org.glassfish.jersey.core:jersey-client:jar:2.27:compile
> [INFO] | | +-
> org.glassfish.jersey.media:jersey-media-jaxb:jar:2.27:compile
> [INFO] | | \- javax.validation:validation-api:jar:1.1.0.Final:compile
> [INFO] | \- javax.ws.rs:javax.ws.rs-api:jar:2.1:compile
> ...
> {noformat}
> This may have been an unintended side effect of the
> [KIP-285|https://cwiki.apache.org/confluence/display/KAFKA/KIP-285%3A+Connect+Rest+Extension+Plugin]
> effort, which added the REST extension for Connect.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
