logstash-plugins / logstash-input-redis

Apache License 2.0
31 stars 69 forks source link

Fix: make sure plugin can be stop-ed #87

Closed kares closed 2 years ago

kares commented 3 years ago

Currently not all specs from the suite are run, as some are tagged with redis. These require a real Redis server, no reason not to run them against CI as well.

While getting CI to run the rest of the suite I realized the plugin isn't always interruptible. In case of data_type => channel and data_type => pattern_channel the Redis API used is blocking (simply waits forever until more data comes in) thus the change to use an alternative "timeout" APIs with these.