-
-
Notifications
You must be signed in to change notification settings - Fork 202
BE: Chore: Upgrade kafka to confluent 7.8.0 #710
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
Conversation
d14e788 to
ffa161f
Compare
9bbf455 to
3726917
Compare
83d88e6 to
5c1a7e5
Compare
|
@yeikel this e2e test seem to be failing, could you take a look? |
e9b8c54 to
69a00d0
Compare
@Haarolean I updated the test. The reason it was failing is because the page was returning 2 pages and the test assumed that all the properties are visible within the first page |
e2e-tests/src/main/java/io/kafbat/ui/screens/topics/TopicSettingsTab.java
Show resolved
Hide resolved
|
@yeikel thank you very much for this PR! |
Description
Before this change, we were using both Confluent Artifacts (via
confluent.version) as well as the OSS artifacts (viakafka-clients.versionandkafka.version).With this change, we are upgrading and aligning the versions to use the Confluent Kafka Distribution. This helps us not only to reduce the attack surface but also to simplify the dependency upgrades
As a side effect of this upgrade, we are also extending the retry logic to retry when there is a re-balance in progress as that was previously failing silently as described in https://issues.apache.org/jira/browse/KAFKA-14455
Closes #693
Closes #692
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)
Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)
A picture of a cute animal (not mandatory but encouraged)