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

Adds ignore_unmapped option to geo queries #17751

Merged
merged 1 commit into from
Apr 14, 2016
Merged

Adds ignore_unmapped option to geo queries #17751

merged 1 commit into from
Apr 14, 2016

Conversation

colings86
Copy link
Contributor

The change adds a new option to the geo_* queries: ignore_unmapped. If this option is set to false, the toQuery method on the QueryBuilder will throw an exception if the field specified in the query is unmapped. If the option is set to true, the toQuery method on the QueryBuilder will return a MatchNoDocsQuery. The default value is false so the queries work how they do today (throwing an exception on unmapped field)

@jpountz
Copy link
Contributor

jpountz commented Apr 14, 2016

LGTM

The change adds a new option to the geo_* queries: ignore_unmapped. If this option is set to false, the toQuery method on the QueryBuilder will throw an exception if the field specified in the query is unmapped. If the option is set to true, the toQuery method on the QueryBuilder will return a MatchNoDocsQuery. The default value is false so the queries work how they do today (throwing an exception on unmapped field)
@colings86 colings86 merged commit c595322 into elastic:master Apr 14, 2016
@colings86 colings86 deleted the enhance/ignoreUnmappedGeo branch April 14, 2016 14:29
This was referenced Apr 14, 2016
@clintongormley clintongormley changed the title Adds ignore_unmapped option to geo queries Adds ignore_unmapped option to geo queries May 2, 2016
@clintongormley clintongormley added :Search/Search Search-related issues that do not fall into other categories and removed :Query DSL labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants