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

KIP 848: Added support for DescribeConsumerGroup for consumer protocol groups #4922

Open
wants to merge 22 commits into
base: dev_kip848_mock_handler_and_integration_tests
Choose a base branch
from

Conversation

PratRanj07
Copy link
Contributor

This PR intends to add support for calling DescribeConsumerGroups for groups created with the new consumer protocol and add backward compatibility to describe group created with the old Classic Protocol.
The working of the API remains similar. Users will continue to call the existing DescribeConsumerGroups api but internally we will be calling the new ConsumerGroupDescribe api to fetch the results. If we get GRP_NOT_FOUND or UNSUPPORTED_VERSION error, then in that case we will call the old DescribeConsumerGroup api.

emasab and others added 19 commits November 29, 2024 17:39
- Mock handler implementation
- Rename current consumer protocol from generic to classic
- Mock handler with automatic or manual assignment
- More consumer group metadata getters
- Test helpers
- Expedite next HB after FindCoordinator
  doing it with an exponential backoff to avoid tight loops
- Configurable session timeout and HB interval
- Fix mock handler ListOffsets response
  LeaderEpoch instead of CurrentLeaderEpoch
- Integration tests passing with AK trunk
- Improve documentation and KIP 848 specific mock tests
- Add mock tests for unknown topic id
  in metadata request and partial reconciliation
- Make test 0147 more reliable
- Fix test 0106 after HB timeout change
- Exclude test case with AK trunk
- Rename rd_kafka_buf_write_tags to
  rd_kafka_buf_write_tags_empty
- Trivup 0.12.5 can run a KafkaCluster
   directly with KRaft and AK trunk
- Trivup 0.12.6 build with a specific commit
Address comment about using rd_kafka_buf_read_topic_partitions
Asserts on non-nullable fields
@PratRanj07 PratRanj07 requested a review from a team as a code owner December 4, 2024 09:58
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

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