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

feat: make the queue fault tolerant #1981

Merged
merged 1 commit into from
Sep 5, 2023
Merged

Conversation

loicmathieu
Copy link
Member

@loicmathieu loicmathieu commented Aug 28, 2023

The Queue is modified to consume Either<T, DeserializationException>.

On the JDBC queue, each JSON exception (not only for missing type) will return an Either with the deserialization exception that now includes the source (not yet used).

All components that read a queue now check for a deserialization exception, if it exists will log it and skip the message.

@brian-mulier-p brian-mulier-p force-pushed the develop branch 2 times, most recently from 8b19413 to d28b63d Compare August 28, 2023 16:11
@tchiotludo tchiotludo force-pushed the fix/either-deserialization branch from 124eb66 to b25ee33 Compare August 28, 2023 19:56
@loicmathieu loicmathieu force-pushed the fix/either-deserialization branch from b25ee33 to 67cb5d8 Compare August 31, 2023 09:34
@loicmathieu loicmathieu marked this pull request as ready for review August 31, 2023 09:48
@loicmathieu loicmathieu force-pushed the fix/either-deserialization branch from 67cb5d8 to 78b0a2d Compare August 31, 2023 10:35
@loicmathieu loicmathieu force-pushed the fix/either-deserialization branch from 78b0a2d to 984e947 Compare September 4, 2023 08:17
@tchiotludo tchiotludo merged commit a85d0e1 into develop Sep 5, 2023
@tchiotludo tchiotludo deleted the fix/either-deserialization branch September 5, 2023 13:31
tchiotludo added a commit to kestra-io/plugin-kubernetes that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-debezium that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-couchbase that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-scripts that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-malloy that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-pulsar that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-redis that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-azure that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-jdbc that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-kafka that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-spark that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-mqtt that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-nats that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-amqp that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-fs that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-aws that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-gcp that referenced this pull request Sep 5, 2023
tchiotludo added a commit to kestra-io/plugin-fs that referenced this pull request Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants