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

incrby parameter order inconsistent #571

Closed
CBenni opened this issue Dec 12, 2014 · 3 comments
Closed

incrby parameter order inconsistent #571

CBenni opened this issue Dec 12, 2014 · 3 comments

Comments

@CBenni
Copy link

CBenni commented Dec 12, 2014

I encountered this:
r.zincrby("key",5,"member")
Gives an exception, eventhough this is the same order of parameters as redis has.
r.zincrby("key","member",5)
works however.

Comparison: r.zadd("key",5,"member") works the way it is supposed to.

@andymccurdy
Copy link
Contributor

We'll look at this for 3.0 as it would be a major backwards incompatible change for existing users.

@andymccurdy andymccurdy added the 3.0 label Jan 4, 2015
@stevenh
Copy link

stevenh commented Mar 13, 2016

Any news on this, coming from redis or other languages / libs like redigo this is a major gotcha and the long its left to fester the worse it gets.

@andymccurdy
Copy link
Contributor

Fixed in redis-py 3.0

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

3 participants