Skip to content

KAFKA-20444: [1/N] Add TxnOffsetCommit v6 schema (KIP-1319)#22205

Merged
dajac merged 2 commits intoapache:trunkfrom
dajac:KAFKA-20444-pr4a
May 5, 2026
Merged

KAFKA-20444: [1/N] Add TxnOffsetCommit v6 schema (KIP-1319)#22205
dajac merged 2 commits intoapache:trunkfrom
dajac:KAFKA-20444-pr4a

Conversation

@dajac
Copy link
Copy Markdown
Member

@dajac dajac commented May 4, 2026

This patch introduces version 6 of the TxnOffsetCommit API:

  • Adds a new TopicId field at 6+ and bounds the existing Name
    field to 0-5 on both the request and the response.
  • Renames the GenerationId field to GenerationIdOrMemberEpoch on the
    request to reflect its dual semantics under the new consumer group
    protocol (source-only rename; the wire format is positional and
    unchanged).
  • Ensures that the client cannot use version 6 yet.
  • Keeps the new version as unstable.

Reviewers: Sean Quah squah@confluent.io

This patch introduces version 6 of the TxnOffsetCommit API:

* Adds a new `TopicId` field at `6+` and bounds the existing `Name`
  field to `0-5` on both the request and the response.
* Renames the `GenerationId` field to `GenerationIdOrMemberEpoch` on
  the request to reflect its dual semantics under the new consumer
  group protocol (source-only rename; the wire format is positional
  and unchanged).
* Ensures that the client cannot use version 6 yet.
* Keeps the new version as unstable.
@dajac dajac marked this pull request as ready for review May 4, 2026 18:43
@dajac dajac requested a review from squah-confluent May 4, 2026 18:49
Copy link
Copy Markdown
Contributor

@squah-confluent squah-confluent 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 patch! Looks good.

@dajac dajac merged commit 7562044 into apache:trunk May 5, 2026
22 checks passed
@dajac dajac deleted the KAFKA-20444-pr4a branch May 5, 2026 18:57
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