Skip to content

Add IgnoreUnmapped to GeoDistanceQuery, GeoBoundingBoxQuery and GeoPo… #6066

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

Closed
wants to merge 1 commit into from

Conversation

alrz1999
Copy link

@alrz1999 alrz1999 commented Dec 4, 2021

…lygonQuery.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Dec 4, 2021

💚 CLA has been signed

@alrz1999
Copy link
Author

alrz1999 commented Dec 4, 2021

As it mentioned in Elasticsearch documents, these kinds of request can support IgnoreUnmapped which is useful for searching on multiple indices. But as I tried to set that to true I found out it doesn't exist.
GeoBoundingBoxQuery
GeoDistanceQuery
GeoPolygonQuery

@stevejgordon
Copy link
Contributor

Thanks for submitting this @alrz1999. Very much appreciated! Would you be able to retarget this to 7.x? The master branch is going to see some major changes in 8.0 as we move to code generating a lot of these types. It should be largely fixed by that work when we're ready to merge it. These should make their way into a 7.x release ideally. If you don't have time I can pick it up at some point.

@alrz1999 alrz1999 changed the base branch from master to 7.x December 8, 2021 13:10
@alrz1999 alrz1999 changed the base branch from 7.x to master December 8, 2021 13:11
@alrz1999
Copy link
Author

alrz1999 commented Dec 8, 2021

Thanks for reviewing. I opened another pull request to add it on 7.x.

@stevejgordon
Copy link
Contributor

Closing this as merged into 7.x in another PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants