Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

This fixes by with expression usage in the MixedClusterEsqlSpecIT tests

Related to #129734

@idegtiarenko idegtiarenko added >test-failure Triaged test failures from CI Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL v8.19.0 v9.1.0 labels Jun 20, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the needs:risk Requires assignment of a risk label (low, medium, blocker) label Jun 20, 2025
;

toIpInAgg
toIpInAgg#[skip:-8.12.99,reason:StatsNestedExp introduced in v8.13.0]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This relies on #104387 added in 8.13

@idegtiarenko idegtiarenko changed the title Fix toIpInAgg Fix toIpInAgg & toIpInSort Jun 20, 2025
;

toIpInSort
toIpInSort#[skip:-8.13.99,reason:SortByExpression introduced in 8.14]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This relies on #107158 added in 8.14

Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

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

LGTM

@idegtiarenko idegtiarenko added >test Issues or PRs that are addressing/adding tests and removed >test-failure Triaged test failures from CI needs:risk Requires assignment of a risk label (low, medium, blocker) labels Jun 20, 2025
@idegtiarenko idegtiarenko merged commit f2703ce into elastic:main Jun 20, 2025
27 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.19

idegtiarenko added a commit to idegtiarenko/elasticsearch that referenced this pull request Jun 20, 2025
kderusso pushed a commit to kderusso/elasticsearch that referenced this pull request Jun 23, 2025
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jun 25, 2025
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 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants