-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[KIP-848] Error Handling for ConsumerGroupHeartbeat API and Partial acknowledgement flow #4601
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
Closed
pranavrth
wants to merge
31
commits into
dev_kip848_ConsumerGroupHeartbeatAPI_assign_revoke_leave
from
dev_kip848_ConsumerGroupHeartbeatAPI_error_handling
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…4404) * KIP-848 new error codes (partial) * Add topic id to topic_partition_private * Rename rd_kafka_uuid_t to rd_kafka_Uuid_t * [KIP-848] Added new configs group.protocol and group.remote.assignor (#4414) Added new configs group.protocol and group.remote.assignor * [KIP-848] Added topic id to topic_partition_t while reading from buffer (#4416) * Address comments * Address comment * Upgrade vcpkg --------- Co-authored-by: Pranav Rathi <[email protected]>
Make install dependent on libs to avoid errors with parallelism. Don't use passed libdir for librdkafka static dependencies built from source
* Bump openssl version partially to 3.0.12 We will stick to 3.0.x with this PR (LTS version) vcpkg doesn't yet have anything greater than 3.0.8, so this is only a partial upgrade. * Update CHANGELOG.md Co-authored-by: Emanuele Sabellico <[email protected]> * Update CHANGELOG.md Co-authored-by: Emanuele Sabellico <[email protected]> --------- Co-authored-by: Emanuele Sabellico <[email protected]>
…e' into dev_kip848_ConsumerGroupHeartbeatAPI_error_handling
Include new trivup patch version in librdkafka
also adds the possibility to generate pseudo-random Uuids
and response with basic cgrp_consumer flow - Added topic id to topic_partition_t while reading from buffer - Added new methods and way to add topic_id to topic partition private - Added new configs group.protocol and group.remote.assignor - Added ConsumerGroupHeartbeat API Request Contract - Added ConsumerGroupApi Request without handling different cases - Working ConsumerGroupHeartbeat API with proper response - Properly receiving assigned partitions with topic_id from the Response - Added metadata request as well after parsing the response. Separate it out from to the topic partition branch - Added metadata response flow with rko - Updated OffsetFetch to v9 - Removed unrequred fields from ConsumerGroupHeartbeat API to make it work with AK > 3.6 - OffsetFetch working fine. Able to consume. - Changed subsribed list to use correct field - Fixed few memory leaks - Some more memory leak fixes. Added updation to subscribed topics list - Minor changes - [KIP-848] Added new configs group.protocol and group.remote.assignor (#4414) - Added new configs group.protocol and group.remote.assignor - Removed printfs and Updated hardcoded one topic for metadata request - Removed some changes related to removed changes in the protocols - [KIP-848] Added topic id to topic_partition_t while reading from buffer (#4416) - Updating topic name from metadata response for all the requested topic_ids instead of just 1. - Style fixes and fixed skip tag issue in buf_*_topic_partition - Changed variable for next assignment - Added topic name while reading topic partition buffer - Changed variable name from assignments to assignment
- Rebased with master - WIP: assignment, revocation, leave group flow - Remove print statements - Remove print statement left - Separate rd_kafka_cgrp_consumer_assignment_done - Allow changing subscription to empty - Expedite next heartbeat - Static group membership and max poll interval checks - Expedite next heartbeat - Fix existing protocol - Partial implementation of reconciliation and next assignment handling - Uniform tests handling across scripts and KRaft mode - Run tests with group.protocol=consumer and reusable condition to skip mock cluster - Test 0113 partial - Test 0018 - Test 0113 stickyness - Test 0113 complete except regex subscription and u_multiple_subscription_changes(true) - Skip some tests, fix subscription change - Test 0029 exclusion clarified - Debug statements - Introduce current assignment rename rkcg_current_target_assignments to rkcg_target_assignment rename rkcg_next_target_assignments to rkcg_next_target_assignment - change to ConsumerGroupHeartbeat in logs - Add remote assignor to debug log - Fix rd_kafka_buf_write_topic_partitions not using topic ids for comparison
… called with duplicate topic id.
Closing in favor of other PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.