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.

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

ZRANGEBYSCORE .. WITHSCORES has return order reversed #39

Closed
mchang31415 opened this issue Jan 31, 2018 · 3 comments
Closed

ZRANGEBYSCORE .. WITHSCORES has return order reversed #39

mchang31415 opened this issue Jan 31, 2018 · 3 comments
Assignees
Labels
kind/bug This issue is a bug

Comments

@mchang31415
Copy link

YUGABYTE:

127.0.0.1:2017> ZADD MCCHANG_TEST_SET2 123 "asdf"
(integer) 1
127.0.0.1:2017> ZRANGEBYSCORE MCCHANG_TEST_SET2 -inf +inf WITHSCORES
1) "123.000000"
2) "asdf"

REDIS:

127.0.0.1:6379>  ZADD MCCHANG_TEST_SET2 123 "asdf"
(integer) 1
127.0.0.1:6379>  ZRANGEBYSCORE MCCHANG_TEST_SET2 -inf +inf WITHSCORES
1) "asdf"
2) "123"

Jedis expects the second value to be a double, and tries to parse as such, leading to exceptions as

Caused by: java.lang.NumberFormatException: For input string: "387a086f-6463-3b26-ae11-5fc6c54aaacf"
        at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:2043)
        at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
        at java.lang.Double.parseDouble(Double.java:538)
        at java.lang.Double.valueOf(Double.java:502)
        at redis.clients.jedis.BuilderFactory$15.build(BuilderFactory.java:267)
        at redis.clients.jedis.BuilderFactory$15.build(BuilderFactory.java:257)
        at redis.clients.jedis.Response.build(Response.java:61)
        at redis.clients.jedis.Response.get(Response.java:37)
       ...
@kmuthukk kmuthukk changed the title ZRANGEBYSCOREWITHSCORES has return order reversed ZRANGEBYSCORE .. WITHSCORES has return order reversed Jan 31, 2018
@rkarthik007 rkarthik007 added the kind/bug This issue is a bug label Jan 31, 2018
@kmuthukk
Copy link
Collaborator

Thanks @mchang31415 for reporting this issue. We will address this quickly...

@kmuthukk
Copy link
Collaborator

kmuthukk commented Feb 2, 2018

@rahuldesirazu : could you link to the commit id for this fix, and close out the issue?

@rahuldesirazu
Copy link
Contributor

Resolved by 1f2f38d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This issue is a bug
Projects
None yet
Development

No branches or pull requests

4 participants