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.

Sign up for GitHub

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

Jump to bottom

let /l/ endpoint return large batches #14

Open
ennorehling opened this issue Jun 19, 2015 · 2 comments
Open

let /l/ endpoint return large batches #14

ennorehling opened this issue Jun 19, 2015 · 2 comments
Assignees
@ennorehling
Labels
bug

Comments

@ennorehling
Copy link
Owner

Two things need to be addressed: The 4096 byte buffer is probably too small (can we stream the result instead?), and the #define BATCH 16 seesm to be limiting the result set, despite best efforts to prevent that.
@ennorehling
Copy link
Owner Author

additional issue: the values in the datastore are not 0-terminated, the size field must be used.

ennorehling added a commit that referenced this issue Jun 19, 2015
@ennorehling
the values in the datastore are not 0-terminated, issue #14
7bfe015
@ennorehling
Copy link
Owner Author

it seems that the two issues were related, and the bad length of the values was causing the short results? it's fixed now.

@ennorehling ennorehling self-assigned this Jun 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ennorehling ennorehling

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ennorehling