Skip to content

Conversation

carlosdelest
Copy link
Member

Follow up to #130523 and #129763

Fix bad merge in #129763 - there were two tests that should have been removed as well. Again, sorry for the trouble.

@carlosdelest carlosdelest added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/Compute Engine Analytics in ES|QL v9.2.0 labels Jul 7, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@nielsbauman nielsbauman left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@carlosdelest carlosdelest enabled auto-merge (squash) July 7, 2025 15:15
@GalLalouche
Copy link
Contributor

FYI, I've unmuted the test from issue #130122 in #130734.

@carlosdelest carlosdelest merged commit dc19728 into elastic:main Jul 7, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Compute Engine Analytics in ES|QL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants