Skip to content

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

Merged
elasticsearchmachine merged 4 commits intoelastic:9.0from
svilen-mihaylov-elastic:backport/9.0/pr-121787
Mar 18, 2025
Merged

[9.0] Added optional parameters to QSTR ES|QL function (#121787)#125114
elasticsearchmachine merged 4 commits intoelastic:9.0from
svilen-mihaylov-elastic:backport/9.0/pr-121787

Conversation

@svilen-mihaylov-elastic
Copy link
Copy Markdown
Contributor

@svilen-mihaylov-elastic svilen-mihaylov-elastic commented Mar 18, 2025

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
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2025

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
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2025

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
Copy Markdown
Contributor Author

svilen-mihaylov-elastic commented Mar 18, 2025

@elasticsearchmachine test this

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2025

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
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2025

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

Reviewers

No reviews

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@svilen-mihaylov-elastic @elasticsearchmachine