-
Couldn't load subscription status.
- Fork 307
Add drop_error_types config option to not retry after certain error types #1228
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
Conversation
26e7204 to
e8afdf4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please do a minor version bump to the gemspec and add an entry to the changelog
Co-authored-by: João Duarte <[email protected]>
Co-authored-by: João Duarte <[email protected]>
Co-authored-by: João Duarte <[email protected]>
Co-authored-by: João Duarte <[email protected]>
3595ff6 to
bc41010
Compare
Co-authored-by: João Duarte <[email protected]>
Co-authored-by: João Duarte <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉 ship it!
This PR adds a config option
drop_error_typesthat allows a user to set a list of error types for which a failed bulk request should not be retried and the events should be dropped. The events are not added to the DLQ. A warn message is logged, if the error type is not listed in the config optionsilence_errors_in_log.Closes #1225