-
Notifications
You must be signed in to change notification settings - Fork 686
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
option to sort by other fields #110
Comments
I wanted to do a spike and see if bleve will work for my client, but they definitely need sorting. At the moment they're only really sorting on "price", which is a float. |
Some interesting bits about how this is done in Lucene: http://blog.trifork.com/2011/10/27/introducing-lucene-index-doc-values/ |
We would love to have sort functionality on other fields as well (for pagination and etc.) |
pagination without sorting is a bit lacking.. is this feature planned anytime soon? |
No one is currently working on it. At the moment I'm focused exclusively on performance, as for our use case sorting/pagination by score is sufficient, but indexing throughput is too low to be usable. Its not a small job, but its not impossible for someone else to take a look at it in parallel. There are two main pieces.
|
@mschoch is anyone working on this currently? I would like to take a stub at this as we need sort for other fields. |
@slavikm I saw you had one collector commit, is this something you're actively working on? |
@caseymrm I started, then had to stop for a while (other stuff came up) and now I'm getting back to it. Hopefully will try and commit something next week. |
really need this feature, hope it will be implemented soon. thank you! @slavikm . |
👍 as a need to be able to use the lib |
Yes, I understand this is important. We will prioritize this and make sure we at least get the API to do this into 1.0. |
I love to see the work here! :D |
Sometimes its useful for applications to want the search results sorted by another field, often a date or numeric value.
The text was updated successfully, but these errors were encountered: