Skip to content

Conversation

luigidellaquila
Copy link
Contributor

Off by one, due to #116964

@craigtaverner I think the PR was supposed to be backported (the label is there at least), but apparently the bot didn't do it.
In case, we should fix it in the 8.x PR as well.

Fixes: #117862

@luigidellaquila luigidellaquila added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL labels Dec 5, 2024
@elasticsearchmachine elasticsearchmachine added v9.0.0 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Dec 5, 2024
@elasticsearchmachine
Copy link
Collaborator

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

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

@luigidellaquila luigidellaquila merged commit 14d9e7c into elastic:main Dec 5, 2024
16 checks passed
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Dec 16, 2024
elasticsearchmachine pushed a commit that referenced this pull request Dec 16, 2024
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this pull request Dec 16, 2024
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 v9.0.0

3 participants