miniredis icon indicating copy to clipboard operation
miniredis copied to clipboard

Published 20 hours ago • verified publisher icon alicebob

LPOS is not supported command for LISTs

Open bpatel85 opened this issue 3 years ago • 2 comments

LPOS is the command to return the index of matching elements inside a Redis list. It doesn't seem to supported at the moment. https://redis.io/commands/lpos

Wondering whether there are plans to add it.

bpatel85 avatar Feb 02 '22 19:02 bpatel85

LMPOP, BLMOVE and BLMPOP are some other that I discovered are not supported.

bpatel85 avatar Feb 02 '22 19:02 bpatel85

Thanks for your issue!

Available since 6.0.6.

that explains, it got added after we added the other L commands, and since then no-one asked for it.

There are no concrete plans, but they are likely not hard to add, and I will have a look when I have some time. PRs are always welcome as well, of course.

alicebob avatar Feb 03 '22 07:02 alicebob

In progress https://github.com/alicebob/miniredis/compare/master...ianstarz:miniredis:lpos?expand=1

I should be able to finish this up by next week.

ianstarz avatar Aug 05 '22 21:08 ianstarz

Ready for review

ianstarz avatar Aug 06 '22 19:08 ianstarz