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

Conversation

Projects
None yet
3 participants
@colings86
Copy link
Member

colings86 commented Apr 14, 2016

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

This comment has been minimized.

Copy link
Contributor

jpountz commented Apr 14, 2016

LGTM

Adds ignore_unmapped option to geo queries
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 force-pushed the colings86:enhance/ignoreUnmappedGeo branch to c595322 Apr 14, 2016

@colings86 colings86 merged commit c595322 into elastic:master Apr 14, 2016

1 check passed

CLA Commit author has signed the CLA
Details

@colings86 colings86 deleted the colings86:enhance/ignoreUnmappedGeo branch Apr 14, 2016

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.