Skip to content

[KIP-932]: segregate acquired records #5257

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.

Sign up for GitHub

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

Jump to bottom
Open
PratRanj07 wants to merge 1 commit into
base: dev_kip-932_share-session-management
Choose a base branch
from
Open

[KIP-932]: segregate acquired records #5257

PratRanj07 wants to merge 1 commit into from

Conversation

@PratRanj07
Copy link
Contributor

@PratRanj07 PratRanj07 commented Dec 2, 2025

This PR intends to:

  • Segregate the acquired records based on the records and the AcquiredRecords array
  • Implement basic implicit acknowledgement(Accept/Gap)
  • Add tests to ensure we are consuming all the messages in one batch only that we get with a ShareFetch response

@PratRanj07 PratRanj07 requested a review from a team as a code owner December 2, 2025 08:48
@confluent-cla-assistant
Copy link

confluent-cla-assistant bot commented Dec 2, 2025

🎉 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.

@PratRanj07 PratRanj07 changed the title Dev kip 932 segregate acquired records [KIP-932]: segregate acquired records Dec 2, 2025
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_kip-932_share-session-management branch from 6c6d281 to 19f1b8d Compare December 12, 2025 14:08
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_kip-932_segregate_acquired_records branch from 6a9d6d3 to ddfb47a Compare December 14, 2025 21:28
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_kip-932_segregate_acquired_records branch from ddfb47a to 29f9770 Compare December 14, 2025 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@PratRanj07