Skip to content
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

KAFKA-16448: Add ProcessingExceptionHandler in Streams configuration #16188

Conversation

loicgreffier
Copy link
Contributor

@cadonna Reopening #16092 after it has been revert for Apache Kafka v3.8.0.


This PR is part of KAFKA-16448 which aims to bring a ProcessingExceptionHandler to Kafka Streams in order to deal with exceptions that occur during processing.

This PR brings ProcessingExceptionHandler in Streams configuration.

Jira: https://issues.apache.org/jira/browse/KAFKA-16448.

Contributors

@Dabz
@sebastienviale
@loicgreffier

Depends On

#16187

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@loicgreffier
Copy link
Contributor Author

loicgreffier commented Jun 7, 2024

@cadonna Rebased this branch on trunk and squashed commits

@loicgreffier loicgreffier force-pushed the KAFKA-16448-Add-Processing-Exception-Handler-StreamsConfig branch 2 times, most recently from 34ba637 to f2d85f8 Compare June 7, 2024 17:33
@loicgreffier loicgreffier force-pushed the KAFKA-16448-Add-Processing-Exception-Handler-StreamsConfig branch from f2d85f8 to 7764007 Compare June 7, 2024 17:37
@cadonna cadonna merged commit 2533a07 into apache:trunk Jun 10, 2024
1 check failed
gongxuanzhang pushed a commit to gongxuanzhang/kafka that referenced this pull request Jun 12, 2024
…pache#16188)

This PR is part of KAFKA-16448 which aims to bring a ProcessingExceptionHandler to Kafka Streams in order to deal with exceptions that occur during processing.

This PR brings ProcessingExceptionHandler in Streams configuration.

Co-authored-by: Dabz <[email protected]>
Co-authored-by: sebastienviale <[email protected]>

Reviewer: Bruno Cadonna <[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.

3 participants