Skip to content
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

Does not work on older versions of Redis #56

Open
AKhoo opened this issue Feb 1, 2023 · 1 comment
Open

Does not work on older versions of Redis #56

AKhoo opened this issue Feb 1, 2023 · 1 comment

Comments

@AKhoo
Copy link

AKhoo commented Feb 1, 2023

sidekiq-grouping does not appear to work for older versions of Redis.

Specifically, one issue I am experiencing is that Redis versions <6.2 don't have the count argument available in lpop, which causes the PLUCK_SCRIPT used by Sidekiq::Grouping::Redis#pluck to result in the following error message:

Redis::CommandError: ERR Error running script (call to f_717b4b61a9929bb9dffcebe0a6697c98b90f6b11): @user_script:1: @user_script: 1: Wrong number of args calling Redis command From Lua script
from /Users/user_name/.rvm/gems/ruby-2.7.3/gems/redis-4.4.0/lib/redis/client.rb:149:in `call'
@russointroitoa
Copy link
Contributor

Hey @AKhoo The issue should be fixed by #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants