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.

Sign up for GitHub

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

Jump to bottom

ZRANGEBYSCORE doesn't support +INF/-INF #326

Closed
kakserpom opened this issue Sep 24, 2022 · 1 comment · Fixed by #336
Closed

ZRANGEBYSCORE doesn't support +INF/-INF #326

kakserpom opened this issue Sep 24, 2022 · 1 comment · Fixed by #336
Assignees
@boazsade
Labels
bug Something isn't working

Comments

@kakserpom
Copy link

To Reproduce
"ZRANGEBYSCORE" "key" "31" "+INF" "WITHSCORES"
(error) ERR min or max is not a float

Expected behavior
Redis supports this.
@kakserpom kakserpom added the bug Something isn't working label Sep 24, 2022
boazsade added a commit to boazsade/dragonfly that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive d…
5bd1a20 
…ragonflydb#326

Signed-off-by: Boaz Sade <[email protected]>
@boazsade
Copy link
Contributor

This issue relate to the use of INF in capital letters (same command with lower case would work)

@boazsade boazsade mentioned this issue Sep 29, 2022
Merged
boazsade added a commit to boazsade/dragonfly that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive d…
ad80d8c 
…ragonflydb#326

Signed-off-by: Boaz Sade <[email protected]>
boazsade added a commit to boazsade/dragonfly that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive d…
389d055 
…ragonflydb#326

Signed-off-by: Boaz Sade <[email protected]>
boazsade added a commit to boazsade/dragonfly that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive d…
6393ea0 
…ragonflydb#326

Signed-off-by: Boaz Sade <[email protected]>
boazsade added a commit to boazsade/dragonfly that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive d…
6a5cbac 
…ragonflydb#326

Signed-off-by: Boaz Sade <[email protected]>
romange pushed a commit that referenced this issue Sep 29, 2022
@boazsade
feat(server): zrange family command support INF as case insensitive #326
7715ee0 
 (#336)

Signed-off-by: Boaz Sade <[email protected]>

Signed-off-by: Boaz Sade <[email protected]>
Co-authored-by: Boaz Sade <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@boazsade boazsade

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(server): zrange family command support INF as case insensitive #326 boazsade/dragonfly
2 participants
@kakserpom @boazsade