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 Kakfa client version to 2.6.0 #456

Merged
merged 1 commit into from
Jan 24, 2022

Conversation

sarunask
Copy link
Contributor

Upgrade Kakfa client version to 2.6.0 as older versions do not work correctly with RedPanda, Kafka cluster alternative, written in GO.
We use RedPanda as an alternative to Confluent Kafka, due to fact it takes ~300Mb to download and run single fast GO service, compared to 2.5Gb of download and run 3 slow starting Java services, which are required to run Kafka.

Signed-off-by: Sarunas Krisciukaitis <sarunas.krisciukaitis@gmail.com>
@sarunask sarunask force-pushed the feature/upgrade_kafka_client_library branch from bc41cd7 to 1a5a69f Compare November 30, 2021 13:26
@yesnault
Copy link
Member

yesnault commented Dec 6, 2021

Hi @sarunask
The min version compatible is V0_10_2_0. You can set your version manually in your testsuite.
Force the default value to another value is a breaking change for all existing users. Why not let user set manually the version to 2.6 ?

@sarunask
Copy link
Contributor Author

sarunask commented Dec 8, 2021

Hi @sarunask The min version compatible is V0_10_2_0. You can set your version manually in your testsuite. Force the default value to another value is a breaking change for all existing users. Why not let user set manually the version to 2.6 ?

I suppose no-one is using v0.10 of Kafka cluster anymore for few years, so it's quite obsolete version. It has bugs and security issues, which are not fixed.
V2.6 can be safe default for upcoming years, as all new Kafka clusters are at least this version or even newer, as now v3.1 is already available.

@fsamin fsamin merged commit bd8fcad into ovh:master Jan 24, 2022
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.

4 participants