Deprecate and remove geo_bounding_box query type parameter

[iverase]

The following parameter is documented in the geo_bounding_box query:

https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-geo-bounding-box-query.html#geo-bbox-type

It seems that is coming from the implementation previous to the introduction of there BKD tree in Lucene. It is currently ignored and it has no effect on the result of there query. The documentation should be updated and the parameter deprecated.

[elasticmachine]

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

[wchaparro]

@iverase should we update the docs for this one? @jrodewig FYI

[jrodewig]

Thanks @wchaparro @iverase.

I've opened a couple of PRs to document the deprecation:

• 8.0: [DOCS] Remove geo_bounding_box query's type parameter #74008
• 7.14: [DOCS] Deprecate geo_bounding_box query's type param (#74009) #74009

These PRs assume we'll deprecate the parameter in 7.14 and remove it in 8.0.

[missinglink]

Hi, can someone please clarify when setting geo_bounding_box.type="indexed" became a no-op?

The reference for 7.0 (for instance) says that the default setting is "memory".

I'd like to better understand what effect removing geo_bounding_box.type="indexed" from our configs will have on users of different versions in the 7.x range.