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

credis_keys returns -97 #25

Closed
GoogleCodeExporter opened this issue Apr 23, 2016 · 6 comments
Closed

credis_keys returns -97 #25

GoogleCodeExporter opened this issue Apr 23, 2016 · 6 comments

Comments

@GoogleCodeExporter
Copy link

I'm using credis 0.2.3 on redis 2.0.0. my problem: credis_keys returns -97!

char **v;
int  ret;

ret = credis_keys( redis, "name:*", &v );

--> ret = -97

Original issue reported on code.google.com by romf...@gmail.com on 5 Sep 2010 at 7:20

@GoogleCodeExporter
Copy link
Author

Hi romfelt, I've asked Salvatore on twitter (@antirez) if there has something 
changed in version 2.0. Our communication:

My question:
@antirez hi salvatore, i'm using credis 0.2.3 on redis 2.0.0. my problem: 
credis_keys returns -97. has keys implementation changed on 2.0?

@kappeTW yep Redis used to return a string with space separated keys, now it 
returns a multi bulk. Did you tried using "hiredis" instead?

I think this will help You to solve the problem!

Best regards

Kappe


Original comment by kappelba...@gmail.com on 6 Sep 2010 at 9:26

@GoogleCodeExporter
Copy link
Author

ok, thanks for the information. it should be a trivial fix. but the question is 
how to treat the different server APIs. Credis is aware of the version of the 
server it is communicating so it is possible that credis have one API that 
handles different versions of the Redis server when possible... 

Original comment by romf...@gmail.com on 6 Sep 2010 at 11:17

@GoogleCodeExporter
Copy link
Author

I thought as much ;-)
perhaps you can publish a "special" credis version for redis 2.0. if not, would 
you be
good enough to send me a patch first of all?

Kappe


Original comment by kappelba...@gmail.com on 6 Sep 2010 at 11:48

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r47.

Original comment by romf...@gmail.com on 6 Sep 2010 at 5:33

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

made credis aware of server version so that a client application doesn't have 
to bother

Original comment by romf...@gmail.com on 6 Sep 2010 at 5:35

@GoogleCodeExporter
Copy link
Author

Good job! Thanks

Original comment by kappelba...@gmail.com on 6 Sep 2010 at 5:55

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

No branches or pull requests

1 participant