Skip to content

Reset errors_count after successful execute()#80

Open
mrkschan wants to merge 1 commit into
logstash-plugins:mainfrom
mrkschan:patch-1
Open

Reset errors_count after successful execute()#80
mrkschan wants to merge 1 commit into
logstash-plugins:mainfrom
mrkschan:patch-1

Conversation

@mrkschan

Copy link
Copy Markdown

After an intermittent issue is resolved in the pipeline, the errors_count is not reset even after a future successful execution.
Eventually, circuit breaker to stop accepting future request.

This patch makes circuit breaker accept future request after the intermittent issue is resolved.

After an intermittent issue is resolved in the pipeline, the `errors_count` is not reset even after a future successful execution.
Eventually, circuit breaker to stop accepting future request.

This patch makes circuit breaker accept future request after the intermittent issue is resolved.
@jordansissel

Copy link
Copy Markdown
Contributor

This plugin has been replaced by the beats input, though I am not sure if this fact is documented anywhere.

@mrkschan

Copy link
Copy Markdown
Author

i think it's the logstash forwarder being replaced but not the working lumberjack protocol. and, this plugin has recent release :) why not keep this improved?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants