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

SAMZA-2617: Upgrade Kafka Client to 2.3.1 #1462

Merged
merged 1 commit into from Mar 3, 2021

Conversation

perkss
Copy link
Contributor

@perkss perkss commented Feb 11, 2021

Issues: The current Kafka version sits at 2.0.1 migrated up to 2.3.1. We get all the bug fixes, enhancements and security upgrades from the Kafka community.
2.4.1 is highest we can go to as it is the last supporting version of Scala 2.11. Done this partial upgrade as going higher TestZkLocalApplication runner becomes flaky and need to investigate why. Want the benefits of 2.3.1 to begin as first step.

Changes: Upgrade Kafka version to 2.3.1. Use TopicPartition as TopicAndPartition will be removed by Kafka 2.4.
Tests: Run the check-all tests.
API Changes: KafkaSystemConsumer protected method toTopicAndPartition removed as no longer required.
Upgrade Instructions: None
Usage Instructions: None

@perkss
Copy link
Contributor Author

perkss commented Feb 12, 2021

@mynameborat @nickpan47 @xinyuiscool @sborya do you think you could take a long at some point please?

@mynameborat
Copy link
Contributor

I am oOO. @Sanil15 can you take point on this PR?

Copy link
Contributor

@Sanil15 Sanil15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this patch

@Sanil15
Copy link
Contributor

Sanil15 commented Mar 1, 2021

@sborya Quick clarification, the Kafka version here is changed from 2.0 to 2.3 and looks like kafka 2.3 even though is compatible but later 2.6 has back incompact, as an extra pre caution imo we should bump samza version here ? what do you think

cc: @perkss

@Sanil15
Copy link
Contributor

Sanil15 commented Mar 3, 2021

Confirmed with boris, that we bump major version to 1.6 as soon as we cut 1.6 branch

@Sanil15 Sanil15 merged commit 89b71ed into apache:master Mar 3, 2021
@akgoel-mo
Copy link

@Sanil15 Any ETA on the next Samza version release which would have the changes from this PR?

We are really looking forward to the ZSTD compression support which kafka client v2.1.0 brings in.

@akgoel-mo
Copy link

@Sanil15 @mynameborat - Any info would be really helpful here.

tranjith pushed a commit to tranjith/samza that referenced this pull request Mar 23, 2021
ehoner pushed a commit to ehoner/samza that referenced this pull request Apr 11, 2023
SAMZA-2617: Upgrade Kafka Client to 2.3.1 (apache#1462)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants