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

fix for the error - wrong number of arguments for expire command #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

raveenb
Copy link

@raveenb raveenb commented Dec 3, 2022

I suspect this issue was due to the underlying library's api signature change

this was tested on

python v3.10,
redis-stack-server c6.2.7
redis-py v4.3.5

@sobolevn
Copy link
Member

sobolevn commented Dec 3, 2022

@skhaz
Copy link

skhaz commented Sep 6, 2023

@sobolevn, I'm also experiencing this issue. Is there anything I can do to help?

@sobolevn
Copy link
Member

sobolevn commented Sep 6, 2023

@skhaz I don't know :(

My solution was to update Redis.
If you find an alternative algorithm to support older versions, please feel free to send a PR.

@skhaz
Copy link

skhaz commented Sep 6, 2023

Got it, those flags, like NX which was used, are an exclusive feature of Redis 7. Since I am using UpStash, they only support up to version 6 at the moment.

Thank you for the library, I'll use it when they release Redis 7.

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

Successfully merging this pull request may close these issues.

3 participants