Skip to content

Conversation

craigtaverner
Copy link
Contributor

This was fixed last week in #125802. It seems this report was from the day before the fix, but the PR was based on main without the mute, so a timing issue. I'll make a PR that simply unmutes the test.

Fixes #125735

@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests :Analytics/Geo Indexing, search aggregations of geo points and shapes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Apr 3, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine merged commit 1645711 into elastic:main Apr 3, 2025
17 checks passed
@craigtaverner craigtaverner deleted the unmute_125735 branch April 3, 2025 11:08
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
This was fixed last week in
elastic#125802. It seems this
report was from the day before the fix, but the PR was based on main
without the mute, so a timing issue. I'll make a PR that simply unmutes
the test.

Fixes elastic#125735
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!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SpatialExtentAggregationNoLicenseIT testStExtentAggregationWithPoints failing

3 participants