Skip to content

[8.17] [8.19] Fixes flaky ST_CENTROID_AGG tests (#114892) (#128037)#128039

Merged
elasticsearchmachine merged 1 commit intoelastic:8.17from
craigtaverner:backport/8.17/pr-128037
May 12, 2025
Merged

[8.17] [8.19] Fixes flaky ST_CENTROID_AGG tests (#114892) (#128037)#128039
elasticsearchmachine merged 1 commit intoelastic:8.17from
craigtaverner:backport/8.17/pr-128037

Conversation

@craigtaverner
Copy link
Copy Markdown
Contributor

Backports the following commits to 8.17:

…8037)

* Fixes flaky ST_CENTROID_AGG tests (elastic#114892)

Even with Kahan summation, we were occasionally getting floating point differences at the 14th decimal point, well beyond anything a GIS use case would care about.

* Unmute after cherry-pick
@craigtaverner craigtaverner added :Analytics/ES|QL AKA ESQL :Analytics/Geo Indexing, search aggregations of geo points and shapes auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels May 12, 2025
@elasticsearchmachine elasticsearchmachine merged commit 5e3174b into elastic:8.17 May 12, 2025
15 checks passed
@craigtaverner craigtaverner deleted the backport/8.17/pr-128037 branch May 12, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL :Analytics/Geo Indexing, search aggregations of geo points and shapes auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.17.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants