Skip to content

[9.0] Added optional parameters to QSTR ES|QL function (#121787) #125114

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Mar 18, 2025
Merged

[9.0] Added optional parameters to QSTR ES|QL function (#121787) #125114

merged 4 commits into from
Mar 18, 2025

Conversation

svilen-mihaylov-elastic
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation
@svilen-mihaylov-elastic
Added optional parameters to QSTR ES|QL function (elastic#121787)
f5c4bd1 
Adds options to QSTR function. elastic#118619 added named function parameters. This PR uses this mechanism for allowing query string function parameters, so query string parameters can be used in ES|QL. Closes elastic#120933 (cherry picked from commit ee4bcac) # Conflicts: #	x-pack/plugin/esql/qa/testFixtures/src/main/resources/qstr-function.csv-spec #	x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/analysis/AnalyzerTests.java
@svilen-mihaylov-elastic svilen-mihaylov-elastic mentioned this pull request Mar 18, 2025
Merged
@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.
@svilen-mihaylov-elastic svilen-mihaylov-elastic added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Mar 18, 2025
@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.
@svilen-mihaylov-elastic
Copy link
Contributor Author

@elasticsearchmachine test this
@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.
@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.
@elasticsearchmachine elasticsearchmachine merged commit 8427114 into elastic:9.0 Mar 18, 2025
15 of 16 checks passed
@svilen-mihaylov-elastic svilen-mihaylov-elastic deleted the backport/9.0/pr-121787 branch March 18, 2025 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v9.0.1

2 participants

@svilen-mihaylov-elastic @elasticsearchmachine