Skip to content

Conversation

@alex-spies
Copy link
Contributor

These were muted due to #131509, which was fixed in #132455.

Let's unmute, so we don't fly blind with respect to inconsistent optimizations and have the generative tests catch them again.

@alex-spies alex-spies added >test Issues or PRs that are addressing/adding tests auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL v9.2.1 v9.3.0 labels Oct 23, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 23, 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!

@alex-spies alex-spies added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 23, 2025
Copy link

@cimequinox cimequinox left a comment

Choose a reason for hiding this comment

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

Removes "optimized incorrectly due to missing references" from ALLOWED_ERRORS as described.

Elasticsearch Serverless Checks did not pass due to what appears to be same unrelated infrastructure issue seen today in another PR #137006 (review)

@alex-spies
Copy link
Contributor Author

Thanks folks!

Unmuting this actually uncovered an issue; this is blocking on #137019, I'm in the process of fixing this.

@alex-spies alex-spies enabled auto-merge (squash) October 29, 2025 09:16
@alex-spies alex-spies merged commit 330c666 into elastic:main Oct 29, 2025
34 checks passed
@alex-spies alex-spies deleted the reenable-gen-tests-with-wrong-optimization branch October 29, 2025 10:29
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-backport Automatically create backport pull requests when merged 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.2.1 v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants