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

Upgrade to Kafka 2.0.0 and Zookeeper 3.4.13 #197

Merged
merged 14 commits into from
Nov 28, 2018
Merged

Upgrade to Kafka 2.0.0 and Zookeeper 3.4.13 #197

merged 14 commits into from
Nov 28, 2018

Conversation

solsson
Copy link
Contributor

@solsson solsson commented Aug 14, 2018

Thanks to @mikenorgate for alerting me on why the zookeeper upgrade is important. We haven't applied this in production yet though.

Edit: Yolean never did the upgrade to 2.0.0 so I didn't merge. Now this PR is instead for Kafka 2.1.0 on Java 11 with images from solsson/dockerfiles#19. It's clearly a destabilization, motivated by the importance of native support for containers in Java.

While destabilizing we will also merge other potentially breaking changes, as tracked by the 5.0 milestone.

@golonzovsky
Copy link

just to mention - Burrow may stop working, see linkedin/Burrow#440

This is the first Kafka release to officially support Java >8.
We should stabilize on it ASAP because of improved docker support in JDK 10+.
See https://blog.docker.com/2018/04/improved-docker-container-integration-with-java-10/
@solsson solsson added this to the 5.0 - Java 11 milestone Nov 28, 2018
solsson added a commit that referenced this pull request Nov 28, 2018
which probably makes the discussion in #112 outdated.
Instead we could discuss -XX:MaxRAMPercentage and related flags from
http://bugs.java.com/view_bug.do?bug_id=JDK-8186248
as soon as they apply any non-testing load, but still
high enough to support for example this repo's eventrouter + tests.
same argument as stated in the 50kafka.yml manifest
@solsson solsson merged commit a521581 into master Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants