search

logstash-plugins / logstash-output-lumberjack

Apache License 2.0
6 stars 25 forks source link

Flush the events when Logstash is shutting down #13

Closed ph closed 8 years ago

ph commented 8 years ago

This plugin has a memory buffer, because it send the events in batch to the lumberjack server, we need to make sure we are flushing the events when we are closing the pipeline

Fixes #12

ph commented 8 years ago

@jsvd I had to reorganize the spec a bit to reuse some of the variables. :(

ph commented 8 years ago

@jsvd I have added back the LogStash::SHUTDOWN check in the #receive method.

jsvd commented 8 years ago

@ph please remove the " " at the end of lines and then :ship: it LGTM

elasticsearch-bot commented 8 years ago

Merged sucessfully into master!