Skip to content

Conversation

@rreddy-22
Copy link
Contributor

@rreddy-22 rreddy-22 commented Apr 9, 2025

This is part of the client side changes required to enable 2PC for
KIP-939

Producer Config:
transaction.two.phase.commit.enable The default would be ‘false’. If
set to ‘true’, the broker is informed that the client is participating
in two phase commit protocol and transactions that this client starts
never expire.

Overloaded InitProducerId method
If the value is 'true' then the corresponding field is set in the
InitProducerIdRequest

Reviewers: Justine Olshan jolshan@confluent.io, Artem Livshits
alivshits@confluent.io

@github-actions github-actions bot added triage PRs from the community producer clients labels Apr 9, 2025
@rreddy-22 rreddy-22 marked this pull request as draft April 14, 2025 16:01
@rreddy-22 rreddy-22 marked this pull request as ready for review April 15, 2025 00:14
@rreddy-22 rreddy-22 changed the title KAFKA-19082: [1/3] Add client config for enable2PC and overloaded initProducerId (KIP-939) KAFKA-19082: [1/4] Add client config for enable2PC and overloaded initProducerId (KIP-939) Apr 15, 2025
@jolshan jolshan added ci-approved and removed triage PRs from the community labels Apr 15, 2025
@rreddy-22 rreddy-22 force-pushed the KAFKA-19082-Client-Side-2PC-Changes-pt1 branch from 0a9fe6a to 5ccb444 Compare April 17, 2025 23:43
Copy link
Contributor

@artemlivshits artemlivshits left a comment

Choose a reason for hiding this comment

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

LGTM

@jolshan jolshan merged commit 62fe528 into apache:trunk Apr 24, 2025
23 checks passed
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.

3 participants