Skip to content
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

WIP: Bump version of kafka-clients to 2.1.1 #90

Closed
wants to merge 1 commit into from

Conversation

sebastianvoss
Copy link

Bump version of kafka-clients to 2.1.1 to get the fix for https://issues.apache.org/jira/browse/KAFKA-7890 (shop-stopper bug when using kafka-clients in environments where brokers can change their IPs, e.g. Kubernetes).

For some reasons test in KafkaAdminClientSpec are red. Maybe we need to wait for embeddedkafka to release a build for Kafka 2.1.1?!

…ues.apache.org/jira/browse/KAFKA-7890 (shop-stopper bug when using kafka-clients in environments where brokers can change their IPs, e.g. Kubernetes)
@vlovgr
Copy link
Contributor

vlovgr commented Feb 22, 2019

Thanks for this @sebastianvoss! In my experience, the embeddedkafka version has to match exactly with the Kafka client version for it to work. Looks like the update has already been merged (#35), but there's no release yet.

As a temporary workaround, 2.1.1 should be binary-compatible with 2.1.0, so if you add it as an explicit dependency in your project, everything should hopefully still work.

@vlovgr
Copy link
Contributor

vlovgr commented Feb 22, 2019

@sebastianvoss Looks like it works if we override the Kafka version in tests. I've branched and included this fix in #91.

@vlovgr vlovgr closed this Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants