Skip to content

GeoBoundingBoxQueryGeoShapeIT should take into account the version #72605

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

Merged
merged 1 commit into from
May 4, 2021

Conversation

iverase
Copy link
Contributor

@iverase iverase commented May 3, 2021

GeoShape legacy parameters should only be used when version is lower than 8.0.

fixes #72602

@iverase iverase added >test Issues or PRs that are addressing/adding tests :Analytics/Geo Indexing, search aggregations of geo points and shapes v8.0.0 labels May 3, 2021
@iverase iverase requested a review from romseygeek May 3, 2021 06:42
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 3, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@imotov imotov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@iverase iverase merged commit 5c6e758 into elastic:master May 4, 2021
@iverase iverase deleted the fixGeoBoundingBoxQueryGeoShapeIT branch May 4, 2021 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] GeoBoundingBoxQueryGeoShapeIT testLimit2BoundingBox failing
4 participants