Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.11] ESQL: Improve agg verification (#99827) #100414

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

alex-spies
Copy link
Contributor

Backports the following commits to 8.11:

When verifying aggregation expressions like
from employees | stats percentile(salary_change, 25*2)
, both arguments are treated the same way during verification. This is
incorrect, as salary_change is the actual aggregation's field, while
25*2 is merely it's (first and only) parameter. This is overly
restrictive.

Apply the current verification only to the aggregation's actual field,
as the parameter is already verified during type resolution (it needs to
be a constant expression).
@alex-spies alex-spies added :Analytics/ES|QL AKA ESQL >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport buildkite-opt-in Opts your PR into Buildkite instead of Jenkins Team:QL (Deprecated) Meta label for query languages team labels Oct 6, 2023
@alex-spies
Copy link
Contributor Author

@elasticsearchmachine run elasticsearch-ci

@alex-spies alex-spies removed the buildkite-opt-in Opts your PR into Buildkite instead of Jenkins label Oct 9, 2023
@alex-spies
Copy link
Contributor Author

Hm, the buildkite CI failed because this does not seem to apply to backports. Since this was only needed as workaround to a Jenkins CI bug on the main PR, and the Jenkins CI seems to have gone fine for this backport, this should be fine to be merged.

@alex-spies alex-spies requested a review from bpintea October 9, 2023 12:38
@alex-spies
Copy link
Contributor Author

@elasticsearchmachine update branch

@alex-spies alex-spies merged commit 1af1dbd into elastic:8.11 Oct 9, 2023
@alex-spies alex-spies deleted the backport/8.11/pr-99827 branch October 9, 2023 12:44
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-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug Team:QL (Deprecated) Meta label for query languages team v8.11.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants