Skip to content

[8.19] ESQL: Extend RENAME syntax to allow a new = old syntax (#129212) #129254

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

Merged
merged 2 commits into from
Jun 11, 2025

Conversation

bpintea
Copy link
Contributor

@bpintea bpintea commented Jun 11, 2025

This extends RENAME's grammar to allow a new syntax: | RENAME new_name = old_name This is supported along the existing ... old_name AS new_name syntax.

Closes #129208

(cherry picked from commit 6c7730d)

…29212)

This extends RENAME's grammar to allow a new syntax: `| RENAME new_name
= old_name` This is supported along the existing `... old_name AS
new_name` syntax.

Closes elastic#129208

(cherry picked from commit 6c7730d)
Copy link
Contributor

Documentation preview:

@@ -11,6 +11,15 @@ The `RENAME` processing command renames one or more columns.
RENAME old_name1 AS new_name1[, ..., old_nameN AS new_nameN]
----

The following syntax is also supported:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@leemthompo, I guess not tagging a specific version is OK here, since we have per-branch docs in 8.x?

@bpintea bpintea added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 11, 2025
@elasticsearchmachine elasticsearchmachine merged commit 30cb238 into elastic:8.19 Jun 11, 2025
15 checks passed
@bpintea bpintea deleted the port/8.19/129212 branch June 11, 2025 13:55
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jun 11, 2025
…astic#129212) (elastic#129254)

* ESQL: Extend `RENAME` syntax to allow a `new = old` syntax (elastic#129212)

This extends RENAME's grammar to allow a new syntax: `| RENAME new_name
= old_name` This is supported along the existing `... old_name AS
new_name` syntax.

Closes elastic#129208

(cherry picked from commit 6c7730d)

* Fix merging
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 >enhancement v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants