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

Add doc values to range fields #24314

Closed
jpountz opened this issue Apr 25, 2017 · 0 comments
Closed

Add doc values to range fields #24314

jpountz opened this issue Apr 25, 2017 · 0 comments
Assignees
@martijnvg
Labels
>enhancement :Search/Search Search-related issues that do not fall into other categories

Comments

@jpountz
Copy link
Contributor

jpountz commented Apr 25, 2017

If we add doc values to range fields and build doc-values-based range queries, then we could leverage IndexOrDocValuesQuery in order to use doc values when they are expected to be more efficient than points.

Related to #23182
@jpountz jpountz added :Search/Search Search-related issues that do not fall into other categories >enhancement labels Apr 25, 2017
@martijnvg martijnvg mentioned this issue May 22, 2017
Merged
martijnvg added a commit to martijnvg/elasticsearch that referenced this issue Jul 12, 2017
@martijnvg
Query range fields by doc values when they are expected to be more ef…
0a25558 
…ficient than points.

* Enable doc values for range fields by default.
* Store ranges in a binary format that support multi field fields.
* Added BinaryDocValuesRangeQuery that can query ranges that have been encoded into a binary doc values field.
* Wrap range queries on a range field in IndexOrDocValuesQuery query.

Closes elastic#24314
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martijnvg martijnvg

Labels
>enhancement :Search/Search Search-related issues that do not fall into other categories
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jpountz @martijnvg