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

Non-integer scores in Zset get rounded to integer in Zrange Withscores #480

Closed
newacct opened this issue Jul 2, 2024 · 1 comment · Fixed by #481
Closed

Non-integer scores in Zset get rounded to integer in Zrange Withscores #480

newacct opened this issue Jul 2, 2024 · 1 comment · Fixed by #481
Labels
bug

Comments

@newacct
Copy link

newacct commented Jul 2, 2024 

ZADD foo 3.14 bar
ZRANGE foo 0 -1 WITHSCORES

returns bar 3
Alex286756 added a commit to atp-mipt/jedis-mock that referenced this issue Jul 3, 2024
@Alex286756
Solution of issue fppt#480
fe8160c
@Alex286756 Alex286756 mentioned this issue Jul 3, 2024
Merged
@inponomarev inponomarev added the bug label Jul 3, 2024
inponomarev pushed a commit to atp-mipt/jedis-mock that referenced this issue Jul 5, 2024
@Alex286756
Solution of issue fppt#480 (#272)
c618c97 
* Solution of issue fppt#480

* Correct for integer scores

* changing the verification

* Update AbstractZRange.java
@inponomarev inponomarev mentioned this issue Jul 5, 2024
Merged
inponomarev added a commit that referenced this issue Jul 5, 2024
@inponomarev @Alex286756
Solution of issue #480 (#272) (#481)
4246c43 
* Solution of issue #480


Co-authored-by: Alex286756 <125192905+Alex286756@users.noreply.github.com>
@inponomarev inponomarev reopened this Jul 5, 2024
@inponomarev
Copy link
Collaborator

Fixed in v1.1.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix rounding of non-integer scores atp-mipt/jedis-mock
2 participants
@newacct @inponomarev