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

Load Filebeat modules pipelines on connect #4479

Merged
merged 1 commit into from
Jun 9, 2017
Merged

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Jun 8, 2017

This is done by registering another callback to the elasticsearch output. We
used to support only one callback, this extends to support multiple callbacks
and also protects the list with a mutex.

There is plan to refactor the package global, but after the publisher refactoring.

Fixes #4444.

@tsg tsg added the review label Jun 8, 2017
This is done by registering another callback to the elasticsearch output. We
used to support only one callback, this extends to support multiple callbacks
and also protects the list with a mutex.

There is plan to refactor the package global, but after the publisher refactoring.

Fixes #4444.
@tsg tsg force-pushed the load_pipeline_on_connect branch from e3415de to 1d398b6 Compare June 8, 2017 15:40
@exekias exekias merged commit 89c5e2d into master Jun 9, 2017
@ruflin
Copy link
Contributor

ruflin commented Jun 11, 2017

@tsg Can the branch be deleted?

@tsg tsg deleted the load_pipeline_on_connect branch June 12, 2017 10:50
@tsg
Copy link
Contributor Author

tsg commented Jun 12, 2017

@ruflin Ooops, must have pushed to the wrong origin, thanks for noticing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants