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

Wrong assertion in SortSupport.java line 50 #18

Closed
sergey-sova opened this issue May 11, 2010 · 3 comments
Closed

Wrong assertion in SortSupport.java line 50 #18

sergey-sova opened this issue May 11, 2010 · 3 comments
Labels

Comments

@sergey-sova
Copy link

Assert.inRange(to, from, Long.MAX_VALUE, "to in LIMIT clause (when from=" + from + ")", ClientRuntimeException.class);

This assertion checking range [from, to], but realy getting range [from, from+to]. So command LIMIT(0, 5) getting 5 elements, command LIMIT(5, 5) getting next 5 elements, but command LIMIT(10, 5) throw exception: org.jredis.ClientRuntimeException: exceeds valid range :{to in LIMIT clause (when from=10)} in method SortSupport.LIMIT() [file: SortSupport.java line:50 - package: org.jredis.ri.alphazero.support]

@alphazero
Copy link
Owner

Thanks for reporting this, asmi. Will take a look.

@alphazero
Copy link
Owner

Fixed:

master (Redis 1.3.x): http://github.com/alphazero/jredis/commit/435512938891234af68851b90e4b528da55fd0df

1.2.n: (Redis 1.2.n):
29c42a7

@sergey-sova
Copy link
Author

Thanks!

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants