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

Enable to monitor batch request failure and retries #211

Closed
wants to merge 5 commits into from

Conversation

simukappu
Copy link
Contributor

@simukappu simukappu commented Nov 28, 2020

Issue #, if available:
#150

Description of changes:

  • Add drop_failed_records_after_batch_request_retries config param to enable you to return failed chunk to Fluentd buffer for retrying
  • Increment num_errors to monitor batch request failure from "monitor_agent" or "fluent-plugin-prometheus" when drop_failed_records_after_batch_request_retries is true (default)
  • Add monitor_num_of_batch_request_retries config param to enable to increment num_errors for each plugin internal retries to monitor batch request retries from "monitor_agent" or "fluent-plugin-prometheus"
  • Including make sleep reliable by measuring how long we actually slept #162

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@simukappu simukappu changed the title Development Enable to monitor batch request failure and retries Nov 28, 2020
@simukappu
Copy link
Contributor Author

We'll close this PR once and merge included commits into master branch. We'll publish this feature as v3.4.0.rc1. Thank you!

@simukappu simukappu closed this Mar 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants