Skip to content

DATAREDIS-756 - Order results of partitioned multi-key commands by positional keys. #303

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

mp911de
Copy link
Member

@mp911de mp911de commented Jan 22, 2018

We now order and assemble results of multi-key commands (e.g. MGET with cross-slot keys) that are executed on different nodes by positional keys to retain duplicate keys in the requested order and retain the server response.

Previously duplicate keys were treated as a set of keys and the response didn't match the requested keys.


Related ticket: DATAREDIS-756.

mp911de and others added 3 commits January 22, 2018 15:09
…sitional keys.

We now order and assemble results of multi-key commands (e.g. MGET with cross-slot keys) that are executed on different nodes by positional keys to retain duplicate keys in the requested order and retain the server response.

Previously duplicate keys were treated as set of keys and the response didn't match the requested keys.
Visibility modifications.
christophstrobl pushed a commit that referenced this pull request Jan 23, 2018
…sitional keys.

We now order and assemble results of multi-key commands (e.g. MGET with cross-slot keys) that are executed on different nodes by positional keys to retain duplicate keys in the requested order and retain the server response.

Previously duplicate keys were treated as set of keys and the response didn't match the requested keys.

Original Pull Request: #303
christophstrobl added a commit that referenced this pull request Jan 23, 2018
Visibility modifications.

Original Pull Request: #303
christophstrobl pushed a commit that referenced this pull request Jan 23, 2018
…sitional keys.

We now order and assemble results of multi-key commands (e.g. MGET with cross-slot keys) that are executed on different nodes by positional keys to retain duplicate keys in the requested order and retain the server response.

Previously duplicate keys were treated as set of keys and the response didn't match the requested keys.

Original Pull Request: #303
christophstrobl added a commit that referenced this pull request Jan 23, 2018
Visibility modifications.

Original Pull Request: #303
christophstrobl pushed a commit that referenced this pull request Jan 23, 2018
…sitional keys.

We now order and assemble results of multi-key commands (e.g. MGET with cross-slot keys) that are executed on different nodes by positional keys to retain duplicate keys in the requested order and retain the server response.

Previously duplicate keys were treated as set of keys and the response didn't match the requested keys.

Original Pull Request: #303
christophstrobl added a commit that referenced this pull request Jan 23, 2018
Visibility changes.

Original Pull Request: #303
@christophstrobl christophstrobl deleted the issue/DATAREDIS-756 branch January 23, 2018 13:30
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