Skip to content

Conversation

mjsax
Copy link
Member

@mjsax mjsax commented Jan 10, 2025

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
producer.abortTransaction() is called. Thus, a
TransactionAbortedException can just be silently swallowed.

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
`producer.abortTransaction()` is called. Thus, a
TransactionAbortedException can just be silently swallowed.
@mjsax mjsax added the streams label Jan 10, 2025
@github-actions github-actions bot added the small Small PRs label Jan 10, 2025
Copy link
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

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

LGTM

@mjsax mjsax merged commit 1672a4b into apache:trunk Jan 15, 2025
10 checks passed
@mjsax mjsax deleted the kafka-18476-transactionabortedexception branch January 15, 2025 02:04
@mjsax
Copy link
Member Author

mjsax commented Jan 15, 2025

Merged to trunk and cherry-picked to 4.0 branch.

mjsax added a commit that referenced this pull request Jan 15, 2025
…18487)

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
`producer.abortTransaction()` is called. Thus, a
TransactionAbortedException can just be silently swallowed.

Reviewers: Bill Bejeck <[email protected]>
pranavt84 pushed a commit to pranavt84/kafka that referenced this pull request Jan 27, 2025
…pache#18487)

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
`producer.abortTransaction()` is called. Thus, a
TransactionAbortedException can just be silently swallowed.

Reviewers: Bill Bejeck <[email protected]>
airlock-confluentinc bot pushed a commit to confluentinc/kafka that referenced this pull request Jan 27, 2025
…pache#18487)

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
`producer.abortTransaction()` is called. Thus, a
TransactionAbortedException can just be silently swallowed.

Reviewers: Bill Bejeck <[email protected]>
manoj-mathivanan pushed a commit to manoj-mathivanan/kafka that referenced this pull request Feb 19, 2025
…pache#18487)

TransactionAbortedException is a follow up error to a previous error,
and such a previous error would already be handled when
`producer.abortTransaction()` is called. Thus, a
TransactionAbortedException can just be silently swallowed.

Reviewers: Bill Bejeck <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants