-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Add support for multiples pipelines #3901
Comments
just for clarification, while a desirable feature to support metrics, multiple pipelines support is not strictly specific for metrics, so in that respect I would remove the "Metrics: " in the subject. Nitpicking, I know. |
@colinsurprenant agree, let me keep at least the metrics label so I can track it :P |
Each pipeline should have an ID to help identify it in the stats. |
The refactor work in #3872 opens up the possibility of multiple pipelines. That PR implements it as an array, but could be an hash, to include the id |
I'm very happy to see this new feature in 2.1 roadmap. |
@purbon I think the initial goal is to run them in isolation, but I agree this could benefit from resource sharing. |
Merged to master |
Currently Logstash support only one pipeline active at any time. The metrics implementation could reuse the pipeline code (input, filter and output) to send events to the metrics backend. We don't want any hiccups of the main pipeline possibly impacting the collection and transport of the metrics.
This changes could also help us do live reload of configuration or cluster management.
Ref #3693 #3774 #3898
The text was updated successfully, but these errors were encountered: