apicurio-registry: Serdes jar does not include all dependencies
The serdes jar file does not include all the dependencies it requires. When running with a sample application that has only org.apache.kafka:kafka-clients:2.5.0 and io.apicurio:apicurio-registry-utils-serde:1.2.2.Final in the dependencies section of the pom.xml I get the following error:
After some experimenting I make more progress by adding javax.enterprise:cdi-api:1.2, org.jboss.resteasy:resteasy-core:4.5.5.Final and org.jboss.resteasy:resteasy-client:4.5.5.Final but the jar file should already have all the dependencies I need, I shouldn’t have to add others to my pom.xml manually.
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 19 (13 by maintainers)
Commits related to this issue
- Add Kafka Serde implementations for Kafka Streams applications - This commit adds three implementations of Kafka's Serde interface which provides a wrapper for a Serializer and Deserializer. The thr... — committed to ajborley/apicurio-registry by ajborley 4 years ago
- Add Kafka Serde implementations for Kafka Streams applications - This commit adds three implementations of Kafka's Serde interface which provides a wrapper for a Serializer and Deserializer. The thr... — committed to ajborley/apicurio-registry by ajborley 4 years ago
- Add Kafka Serde implementations for Kafka Streams applications (#839) - This commit adds three implementations of Kafka's Serde interface which provides a wrapper for a Serializer and Deserializer. ... — committed to Apicurio/apicurio-registry by ajborley 4 years ago
Hi @EricWittmann,
I’m in favour of removing all that JAX-RS classes. I’ll have a look at this and improve this implementation.
You’re not wrong. Ideally that should be the only dependency. For now we’ll have to document the issue (users might need the RE rest client dependency as well, depending on their application). The old client interface that uses the JAX-RS classes has been deprecated - so the next major release should clean the last of this up.