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

[7.x] Adds support for geo-bounds filtering in geogrid aggregations #50996

Merged
merged 2 commits into from Jan 14, 2020

Conversation

@talevy
Copy link
Contributor

talevy commented Jan 14, 2020

It is fairly common to filter the geo point candidates in
geohash_grid and geotile_grid aggregations according to some
viewable bounding box. This change introduces the option of
specifying this filter directly in the tiling aggregation.

This is even more relevant to geo_shape where the bounds will restrict
the shape to be within the bounds

this optional bounds parameter is parsed in an equivalent fashion to
the bounds specified in the geo_bounding_box query.

backport of #50002.

It is fairly common to filter the geo point candidates in
geohash_grid and geotile_grid aggregations according to some
viewable bounding box. This change introduces the option of
specifying this filter directly in the tiling aggregation.

This is even more relevant to `geo_shape` where the bounds will restrict
the shape to be within the bounds

this optional `bounds` parameter is parsed in an equivalent fashion to
the bounds specified in the geo_bounding_box query.
@elasticmachine

This comment has been minimized.

Copy link
Collaborator

elasticmachine commented Jan 14, 2020

Pinging @elastic/es-analytics-geo (:Analytics/Geo)

@talevy

This comment has been minimized.

Copy link
Contributor Author

talevy commented Jan 14, 2020

run elasticsearch-ci/2

talevy added a commit to talevy/elasticsearch that referenced this pull request Jan 14, 2020
talevy added a commit that referenced this pull request Jan 14, 2020
@talevy talevy merged commit 9ee2e11 into elastic:7.x Jan 14, 2020
10 checks passed
10 checks passed
CLA All commits in pull request signed
Details
elasticsearch-ci/1 Build finished.
Details
elasticsearch-ci/2 Build finished.
Details
elasticsearch-ci/bwc Build finished.
Details
elasticsearch-ci/default-distro Build finished.
Details
elasticsearch-ci/docs Build finished.
Details
elasticsearch-ci/docs-check Build finished.
Details
elasticsearch-ci/oss-distro-docs Build finished.
Details
elasticsearch-ci/packaging-sample-matrix-unix Build finished.
Details
elasticsearch-ci/packaging-sample-matrix-windows Build finished.
Details
@talevy talevy deleted the talevy:7x-6c86606d2 branch Jan 14, 2020
talevy added a commit that referenced this pull request Jan 14, 2020
…fter #50996 (#50997)

after #50996 (backport of #50002) merged, the version guard 
on geo-grid `bounds` parameter can be updated to 7.6

re-enables bwc tests
SivagurunathanV added a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
SivagurunathanV added a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
…fter elastic#50996 (elastic#50997)

after elastic#50996 (backport of elastic#50002) merged, the version guard 
on geo-grid `bounds` parameter can be updated to 7.6

re-enables bwc tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.