fix shutdown controller when using no filters

[jsvd]

initializes @filter_threads to be an empty array so that the Shutdown controller can deal with no filters gracefully.

this needs to be merged into master, 2.x and 2.1

fixes #4271

[andrewvc]

Does this need to be merged into master of the new pipeline is to be merged
soon?

On Friday, November 27, 2015, João Duarte notifications@github.com wrote:

initializes @filter_threads to be an empty array so that the Shutdown
controller can deal with no filters gracefully.

this needs to be merged into master, 2.x and 2.1

fixes #4271 #4271

You can view, comment on, or merge this pull request online at:

#4284
Commit Summary

• fix shutdown controller when using no filters

File Changes

• M logstash-core/lib/logstash/pipeline.rb
  https://github.com/elastic/logstash/pull/4284/files#diff-0 (3)
• M logstash-core/spec/logstash/pipeline_spec.rb
  https://github.com/elastic/logstash/pull/4284/files#diff-1 (23)

Patch Links:

• https://github.com/elastic/logstash/pull/4284.patch
• https://github.com/elastic/logstash/pull/4284.diff

—
Reply to this email directly or view it on GitHub
#4284.

[jsvd]

@andrewvc it's not mandatory though I don't see harm in it.

[ph]

LGTM

[andrewvc]

@jsvd I think it would make sense not to merge this into master given that it's just going to create a merge conflict when the new pipeline is merged in. Just my $0.02

[suyograo]

@jsvd can you merge this to 2.1 and 2.x? I will create a release tomorrow