-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does not work with dependency kafka-avro-serializer #29
Comments
It sounds like the Avro serializer library has a transitive dependency on Curator. Try adding an exclusion rule to the dependency declaration. |
Do you have a working example? I'm not sure how to do that. I think that's something that should be documented on your front page as well because I'm sure other people will encounter that issue. And I don't see Curator here unfortunately :(
|
I think the issue is that you're targetting curator-test 4.0.0. This dependency is on Zookeeper 3.5.x, which is not the zookeeper supported by Kafka. Maybe try downgrading to |
Ah, it must be the transitive dependency on Zookeeper. Try excluding that from the Avro dependency. <dependency>
<groupId>io.confluent</groupId>
<artifactId>kafka-avro-serializer</artifactId>
<version>3.3.0</version>
<exclusions>
<exclusion>
<groupId>org.apache</groupId>
<artifactId>zookeeper</artifactId>
</exclusion>
</exclusions>
</dependency> I'd rather not downgrade Curator now because it could cause a lot of existing code to break. |
According to https://mvnrepository.com/artifact/org.apache.curator/curator-test It seems curator-test was upgraded to 4.0.0 as part of this commit: 89bfeed Although I'm not sure if there's any reason why it has. Bottom line, I believe downgrading to 2.12 is both safer and more sound based on matching the kafka zookeeper client version. It also prevents projects onboarding this library from having zookeeper clients in version beta 3.5.x, which could be a serious risk In #30 Jenkins tested the project itself and it passes its own test suite, which is a pretty good indication the downgrade shouldn't break anything I think it's still time to make a good decision for this project to follow the 2.x branch of curator-test. If you don't agree with me that's fine I won't insist :) Hope you understand my point of view |
It's been a while so I can't remember the reason for the upgrade. My fear is that the 3.x version has been out for a long time and by suddenly downgrading one of the dependencies, I could break existing code that may be accidentally (or, indeed, intentionally) relying on it. As this is a minor dependency conflict with a supporting Kafka library, I don't feel like a downgrade is justified here. If you're uncomfortable with the dependency versions, feel free to keep your fork up. The code is Apache licensed and any future updates are most likely to be just Kafka version updates anyway. |
When you include
one gets
Is it possible to get official guidance to make the two libraries co-exist?
The text was updated successfully, but these errors were encountered: