Skip to content

Conversation

@jordan-powers
Copy link
Contributor

@jordan-powers jordan-powers commented Aug 29, 2025

In f2b364c we added another function, which broke this test, which we fixed in #133820.
We then immediately added another function in 97abc87, so let's fix this test again here.

Fixes #133840, Fixes #133841, Fixes #133842

@jordan-powers jordan-powers added >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build :Analytics/ES|QL AKA ESQL v9.2.0 labels Aug 29, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Aug 29, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@luigidellaquila luigidellaquila 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!

Copy link
Contributor

@leontyevdv leontyevdv left a comment

Choose a reason for hiding this comment

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

LGTM!

@jordan-powers jordan-powers enabled auto-merge (squash) September 2, 2025 05:28
@jordan-powers jordan-powers merged commit 2404e24 into elastic:main Sep 2, 2025
34 of 35 checks passed
@jordan-powers jordan-powers deleted the fix-esql-yaml-test branch September 2, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v9.2.0

Projects

None yet

4 participants