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

[admin] Add consumer group bindings for KIP 222, 518, 396 (partial) and DeleteConsumerGroups #923

Merged
merged 2 commits into from
Jan 17, 2023

Conversation

emasab
Copy link
Contributor

@emasab emasab commented Jan 13, 2023

This includes #852 (by @vsantwana) - the DeleteGroups implementation.
The rest of the bindings are - ListConsumerGroups, DescribeConsumerGroups, AlterConsumerGroupOffsets, and ListConsumerGroupOffsets.

@emasab emasab force-pushed the feature/list-and-alter-offsets branch 3 times, most recently from 74675fa to 2c4d900 Compare January 13, 2023 13:57
@emasab emasab marked this pull request as ready for review January 13, 2023 13:58
@emasab emasab changed the title [admin] Add consumer group bindings for KIP-88, 222, 518, 396 (partial) [admin] Add consumer group bindings for 222, 518, 396 (partial) and DeleteConsumerGroups Jan 13, 2023
@emasab emasab changed the title [admin] Add consumer group bindings for 222, 518, 396 (partial) and DeleteConsumerGroups [admin] Add consumer group bindings for KIP 222, 518, 396 (partial) and DeleteConsumerGroups Jan 13, 2023
@milindl
Copy link
Contributor

milindl commented Jan 16, 2023

Added a commit fixing the tests (the tests were not changed while changing the return type of DescribeConsumerGroups)
Additionally, I've also changed the return type of DeleteConsumerGroups (we return now a DeleteConsumerGroupResult rather than a slice of GroupResult) to make it uniform with the other APIs.

milindl and others added 2 commits January 16, 2023 15:34
DescribeConsumerGroups,
ListConsumerGroupOffsets (KIP-222)
DeleteConsumerGroups,
AlterConsumerGroupOffsets (KIP-396),
Allow listing consumer groups per state (KIP-518),

Co-authored-by: Emanuele Sabellico <[email protected]>
Co-authored-by: Santwana Verma <[email protected]>
also contains some documentation, changelog changes (minor).
@emasab emasab force-pushed the feature/list-and-alter-offsets branch from 16f671d to ef0fffa Compare January 16, 2023 14:38
Copy link
Contributor Author

@emasab emasab left a comment

Choose a reason for hiding this comment

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

Great job @milindl @vsantwana! I've moved this to an internal branch and opened the PR, so @milindl go on with approving and merging it.

@milindl milindl self-requested a review January 17, 2023 04:01
@milindl milindl merged commit f56f3d2 into master Jan 17, 2023
@milindl milindl deleted the feature/list-and-alter-offsets branch January 17, 2023 08:10
PrasanthV454 pushed a commit that referenced this pull request Mar 17, 2023
…nd DeleteConsumerGroups (#923)

Implement ListConsumerGroups, DescribeConsumerGroups,
ListConsumerGroupOffsets (KIP-222), DeleteConsumerGroups,
AlterConsumerGroupOffsets (KIP-396),
Allow listing consumer groups per state (KIP-518).

Co-authored-by: Milind L <[email protected]>
Co-authored-by: Santwana Verma <[email protected]>
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