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

SORT command not supported #45

Open
amansardana opened this issue Aug 16, 2018 · 5 comments
Open

SORT command not supported #45

amansardana opened this issue Aug 16, 2018 · 5 comments
Assignees

Comments

@amansardana
Copy link

Hi,
I am trying to write unit tests for our system using miniredis. Our system utilises SORT command of redis extensively. But miniredis doesn't support SORT command yet. I would like to request the addition of SORT command support in the library.
Regards

@alicebob
Copy link
Owner

Thank you for your report!
I have no idea how I missed the SORT command, and it should indeed be added.
On which datatypes do you use the command? list or sets?

@alicebob alicebob self-assigned this Aug 16, 2018
@amansardana
Copy link
Author

I use the SORT primarily with sets.
Just now i noticed that ZREVRANGEBYLEX command is also missing support.

@alicebob
Copy link
Owner

ok. At least that one is relatively new :)

@jpatino-godaddy
Copy link

is this not supported yet?

@alicebob
Copy link
Owner

SORT is one of these messy commands with a ton of options and special cases. If someone makes a PR I'm happy to merge it :)

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

No branches or pull requests

3 participants