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

[Bug] There may be some elements missed in the reply of zrangebyscore/zrevrangebyscore #8

Open
shangxiaoxiong opened this issue Feb 8, 2022 · 5 comments
Open

[Bug] There may be some elements missed in the reply of zrangebyscore/zrevrangebyscore #8

shangxiaoxiong opened this issue Feb 8, 2022 · 5 comments

Comments

@shangxiaoxiong
Copy link

1）Bug

  • zadd 7 members
    image

  • only 3 members can be read
    image

2）Fix
refer to kvrocks/pull/434
@mytrygithub
Copy link
Collaborator

目前分析是todis对于-inf和+inf的处理有问题，处理中

@shangxiaoxiong
Copy link
Author

inf 没处理好，应该是 std::nextafter 这个函数处理不了 inf 的情况，可以看一下 blackwidow 内关于这个函数的调用

@shangxiaoxiong
Copy link
Author

在调用 std::nextafter 的地方对 inf 特殊处理应该就 ok 了

@mytrygithub
Copy link
Collaborator

目前分析是对于-inf和+inf有手动设置处理，但是处理有问题，暂不涉及std::nextafter

@mytrygithub
Copy link
Collaborator

处理zrangbyscore时不涉及std::nextafter，在处理zrevrangebyscore时是std::nextafter无法处理max的情况

This was referenced Feb 16, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mytrygithub @shangxiaoxiong