Support for REDIS ZSET

[amaury-d]

This commit adds support for ZSET queues in REDIS with data_type sortedset.
Batch is supported.

Score of the message is based on output (see logstash-output-redis).

The pull request adds the following option to the configuration:

• priority_reverse boolean, default false, pop high score items
  first

Usage example:

    input {
    	redis {
		host => "127.0.0.1"
		port => 6379
		data_type => "sortedset"
		key => "syslog"
		priority_reverse => true
	}
    }

=> Il will pop from the sortedset with the key syslog the messages with the higher scores first

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

[amaury-d]

This pull request is linked to logstash-plugins/logstash-output-redis#46

[untergeek]

@amaury-d Can you please add some rspec tests for these changes? This is a rather significant update and some tests are absolutely necessary.

[amaury-d]

@untergeek Sure, i'll work on it and update the pull request

[andrewvc]

@amaury-d any updates here on adding those specs? We'd love to merge this :)

[amaury-d]

Hi, I started working on testing but I didn't have time to finish it until now.. I hope to be able to work on it in week 23 to 26.

…

On Fri, Apr 28, 2017 at 5:52 PM, Andrew Cholakian ***@***.***> wrote: @amaury-d <https://github.com/amaury-d> any updates here on adding those specs? We'd love to merge this :) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#56 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABDuXv8y1gZZ-2EaNlEQ99QNXoBiwCY0ks5r0gtCgaJpZM4L5a1C> .

-- Amaury Decrême

[amaury-d]

Hi @untergeek @andrewvc ,

rspec tests and supplementary documentation have been added to pull requests (input and output).

[tandara0]

I really want it to support sortedset data_type.
Please merge this to master.

[tandara0]

Can I resolved the conflicts and merge the code to the master?

[amaury-d]

@tandara0 That's fine for me. This PR seems to have been long forgotten and I'm happy if it can be useful to someone else.

Therefore, I'm not sure how you can work on this PR on github.

[tandara0]

@tandara0 That's fine for me. This PR seems to have been long forgotten and I'm happy if it can be useful to someone else.

Therefore, I'm not sure how you can work on this PR on github.

Thanks. I will try.