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

Concurrent reads and writes might fetch the same values multiple times. #11

Closed
hmalphettes opened this issue Sep 10, 2014 · 2 comments
Closed

Concurrent reads and writes might fetch the same values multiple times. #11

hmalphettes opened this issue Sep 10, 2014 · 2 comments
Assignees
@hmalphettes

Comments

@hmalphettes
Copy link
Owner

Redisdown iterator fetches records using an ordered range query.
At the moment it relies on an offset to get the next page.
If new records are inserted at the beginning of the collection the records that have already been fetched will be fetched again and iterated on multiple times.

Must use the last key retrieved from the previous page to fetch the current page instead of the offset.
@hmalphettes hmalphettes self-assigned this Sep 10, 2014
@hmalphettes
Copy link
Owner Author

@ApeChimp thanks for making me think about possible concurrent write issues.

@hmalphettes hmalphettes mentioned this issue Sep 17, 2014
Closed
@hmalphettes
Copy link
Owner Author

Fixed with this commit: cc575d9

Published as v0.1.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hmalphettes hmalphettes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hmalphettes