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

Random errors (-95, -97,more data needed) using credis_lrange on large lists #4

Closed
GoogleCodeExporter opened this issue Nov 3, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. using credis_lrange(redis,"myList",0,-1,&elements); on large lists

What is the expected output? What do you see instead?
It works most of the time but on every 3./4. call the function returns -95, -97 
or displays: more 
data needed. 

What version of the product are you using? On what operating system?
credis 0.1

Please provide any additional information below.
To receive any data I had to set:

#define CR_BUFFER_SIZE 2048
#define CR_MULTIBULK_SIZE 2048

Original issue reported on code.google.com by rea...@imojo.de on 8 Feb 2010 at 9:46

@GoogleCodeExporter
Copy link
Author

I have just checked this issue with SVN r13 and the random errors seams to be 
gone but if CR_BUFFER_SIZE is set 
to small the program stops with: Program received signal:  “EXC_BAD_ACCESS” 
at credis.c line 106 (if (*(buf++) 
== '\r'))

Original comment by rea...@imojo.de on 8 Feb 2010 at 10:15

@GoogleCodeExporter
Copy link
Author

i found a general bug in both muli-bulk and bulk reception. could you please 
verify 
again with r15

Original comment by romf...@gmail.com on 9 Feb 2010 at 8:26

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Thanks for all your work. This is really a great help!

I will check it for r15.

You can see also issue 11 where I have attached a test script that I use to 
debug the lists.

Original comment by rea...@imojo.de on 9 Feb 2010 at 8:34

@GoogleCodeExporter
Copy link
Author

Thanks a lot. This issue is fixed with r15.

Original comment by rea...@imojo.de on 9 Feb 2010 at 9:00

@GoogleCodeExporter
Copy link
Author

fix verified by reporter

Original comment by romf...@gmail.com on 9 Feb 2010 at 9:09

  • Changed state: Verified

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