-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Create zset index for sorting and range query [DATAREDIS-814] #1387
Comments
Mark Paluch commented Thanks for raising this issue. That sounds interesting. How did you solve duplicate values (i.e. multiple objects have the same numeric value in one field)? Feel free to submit a pull request so we can have a look and continue discussion on code level |
Yan Ma commented Range query returns all the matching records. Pls grant me privilege to new a branch. Thanks |
Mark Paluch commented If you want to contribute, fork the Spring Data Redis repository, push your commits on a branch in your fork and submit a pull request. You can find here our contribution guide |
awso commented My pull request stucks at Travis CI check: Any hints/documents about how to fix it please? https://travis-ci.org/spring-projects/spring-data-redis/builds/572529635 Installing oraclejdk8
|
Mark Paluch commented Let's revisit this topic for Spring Data Redis 3.0 |
Many thanks @mp911de. any question/comment, pls let me know! |
Any update on this? This feature would be a great value addition. |
Yan Ma opened DATAREDIS-814 and commented
We are using Redis as the main in-memory DB. Our business requirements heavily demand range queries based on attributes of timestamp type or numeric type.
We need an easy way to define the indexes that enable range queries such as LessThan, Between and GreaterThan, etc.
Also we need the complex queries that are based on simple indexes and the range query indexes.
I updated the relevant code in index resolving, writing, query and deleting to support above requirements. Would like to contribute the code to spring-data-redis so that more people can benefit from this.
Referenced from: pull request #471
The text was updated successfully, but these errors were encountered: