Skip to content

DATAREDIS-531 - Scan commands should use a dedicated bound connection. #218

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

Closed
wants to merge 3 commits into from

Conversation

christophstrobl
Copy link
Member

SCAN, SSCAN, HSCAN and ZSCAN now reference a dedicated connection that is bound to the Cursor. This allows creation of multiple cursors for different threads without the risk of potentially sharing a single connection. As before the caller is responsible for closing the Cursor correctly after usage.

christophstrobl and others added 3 commits August 31, 2016 12:22
SCAN, SSCAN, HSCAN and ZSCAN now reference a dedicated connection that is bound to the Cursor. This allows creation of multiple cursors for different threads without the risk of potentially sharing a single connection. As before the caller is responsible for closing the Cursor correctly after usage.
Use quit method for close verification in tests. Refine static imports. Fix spelling.
mp911de pushed a commit that referenced this pull request Sep 5, 2016
SCAN, SSCAN, HSCAN and ZSCAN now reference a dedicated connection that is bound to the Cursor. This allows creation of multiple cursors for different threads without the risk of potentially sharing a single connection. As before the caller is responsible for closing the Cursor correctly after usage.

Original pull request: #218.
mp911de added a commit that referenced this pull request Sep 5, 2016
Use quit method for close verification in tests. Refine static imports. Fix spelling.

Original pull request: #218.
@mp911de mp911de closed this Sep 5, 2016
@mp911de mp911de deleted the issue/DATAREDIS-531-alternate branch September 5, 2016 09:32
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

Successfully merging this pull request may close these issues.

2 participants