Skip to content

Remove bounding box query type parameter#74536

Merged
iverase merged 3 commits intoelastic:masterfrom
iverase:removebboxtype
Jun 28, 2021
Merged

Remove bounding box query type parameter#74536
iverase merged 3 commits intoelastic:masterfrom
iverase:removebboxtype

Conversation

@iverase
Copy link
Contributor

@iverase iverase commented Jun 24, 2021

The parameter has been deprecates in 7.14 as it is a no-op.

@iverase iverase added >breaking :Analytics/Geo Indexing, search aggregations of geo points and shapes v8.0.0 labels Jun 24, 2021
@iverase iverase requested review from imotov and jrodewig June 24, 2021 05:54
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jun 24, 2021
@elasticmachine
Copy link
Collaborator

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

@iverase
Copy link
Contributor Author

iverase commented Jun 24, 2021

@elasticmachine run elasticsearch-ci/part-2

Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

Looks good overall, but there's a formatting issue in the breaking changes we'll want to fix before merging. I left a suggestion to fix it.

Thanks @iverase!

Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

Docs LGTM. Thanks @iverase.

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 d7ef5b6 into elastic:master Jun 28, 2021
@iverase iverase deleted the removebboxtype branch June 28, 2021 05:37
Joxit added a commit to pelias/query that referenced this pull request May 6, 2025
The parameter has been deprecates in 7.14 as it is a no-op.

see elastic/elasticsearch#74536
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 >breaking Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants