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

Catch record deserialization exceptions that may happen while polling #771

Merged
merged 2 commits into from
Jul 20, 2023
Merged

Catch record deserialization exceptions that may happen while polling #771

merged 2 commits into from
Jul 20, 2023

Conversation

guillermocalvo
Copy link
Contributor

Fixes #640

@sdelamo sdelamo added the type: bug Something isn't working label Jul 19, 2023
@sdelamo sdelamo requested a review from jeremyg484 July 19, 2023 11:02
@sdelamo sdelamo merged commit c0759e7 into micronaut-projects:master Jul 20, 2023
4 checks passed
@guillermocalvo guillermocalvo deleted the 640-messages-not-reprocessed-by-consumer-when-serializationexception-occurs-while-producing-message branch July 20, 2023 12:31
guillermocalvo added a commit to giamo/micronaut-kafka that referenced this pull request Aug 4, 2023
Previous behavior was buggy. It was fixed by:
micronaut-projects#771
sdelamo pushed a commit that referenced this pull request Aug 18, 2023
* Allow to commit offset in case of serialization errors

* Disable test kafka listeners unless running their specific test

* Add documentation

* Update test to reflect current behavior

Previous behavior was buggy. It was fixed by:
#771

* Create config classes for the default kafka listener exception handler

* Update documentation

* Add an empty, `@Deprecated` constructor

---------

Co-authored-by: Guillermo Calvo <calvog@objectcomputing.com>
Co-authored-by: Guillermo Calvo <guillermo@guillermo.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Messages not reprocessed by consumer when SerializationException occurs while producing message
3 participants