Skip to content

Conversation

lianetm
Copy link
Member

@lianetm lianetm commented Sep 12, 2025

Clarify rebalance callbacks behaviour (got some questions for
onPartitionsAssigned, docs where indeed confusing about the partitions
received in params). Reviewed all rebalance callbacks with it.

Reviewers: Bill Bejeck[email protected]

Copy link
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

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

Nice improvement @lianetm LGTM with a few minor comments.

@github-actions github-actions bot removed the small Small PRs label Sep 12, 2025
@lianetm
Copy link
Member Author

lianetm commented Sep 12, 2025

Thanks for the review @bbejeck ! All comments addressed.
Also added 3 integration tests to showcase the behaviour, and added so more clarifications around it. Please take another look if you have a chance. Thanks a lot!

Copy link
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

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

Thanks for the follow-up @lianetm - LGTM modulo fixing the tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants