dragonfly icon indicating copy to clipboard operation
dragonfly copied to clipboard

feat(server): Support for LIMIT and REV in ZRANGE #422

Open RedhaL opened this issue 2 years ago • 2 comments

Closes #422

Since the new ZRange regroups all the other (now deprecated) ZRange functions, I did some refactoring to avoid duplicating code from these functions. This made it possible to have support for ZRevRangeByLex for free, since the visitor function already supported it. Also spotted and corrected an error in ZRangeByLex when parsing the offset parameter.

There are some unrelated formatting changes that were added by clang-format.

RedhaL avatar Nov 01 '22 22:11 RedhaL

I made a mistake when handling parameters following the LIMIT group, the new commits should fix that

RedhaL avatar Nov 02 '22 09:11 RedhaL

@RedhaL i pinged you at discord :)

romange avatar Nov 02 '22 11:11 romange

@dranikpg I just added the fix for zrangebyscore as well

RedhaL avatar Nov 03 '22 10:11 RedhaL