ESQL: Remove aliasing inside Eval #100174
Assignees
Labels
Comments
|
Pinging @elastic/es-ql (Team:QL) |
elasticsearchmachine
added
the
Team:QL (Deprecated)
|
Pinging @elastic/elasticsearch-esql (:Query Languages/ES|QL) |
costin
added a commit
to costin/elasticsearch
that referenced
this issue
Oct 4, 2023
Evals that introduce aliased can be simplified by extracting them into a project (and thus signaling there's no underlying processing). The following eval: eval x = a + 1, y = x, z = y + 1, y = z, w = y + 1 can be converted into: eval x = a + 1, z = a + 1 + 1, w = a + 1 + 1 | project x, z, z as y, w Fix elastic#100174
costin
added a commit
that referenced
this issue
Oct 5, 2023
Evals that introduce aliased can be simplified by extracting them into a project (and thus signaling there's no underlying processing). The following eval: eval x = a + 1, y = x, z = y + 1, y = z, w = y + 1 can be converted into: eval x = a + 1, z = a + 1 + 1, w = a + 1 + 1 | project x, z, z as y, w Fix #100174 Fix #100050
costin
added a commit
to costin/elasticsearch
that referenced
this issue
Oct 5, 2023
Evals that introduce aliased can be simplified by extracting them into a project (and thus signaling there's no underlying processing). The following eval: eval x = a + 1, y = x, z = y + 1, y = z, w = y + 1 can be converted into: eval x = a + 1, z = a + 1 + 1, w = a + 1 + 1 | project x, z, z as y, w Fix elastic#100174 Fix elastic#100050
elasticsearchmachine
pushed a commit
that referenced
this issue
Oct 5, 2023
* ESQL: Remove aliasing inside Eval (#100238) Evals that introduce aliased can be simplified by extracting them into a project (and thus signaling there's no underlying processing). The following eval: eval x = a + 1, y = x, z = y + 1, y = z, w = y + 1 can be converted into: eval x = a + 1, z = a + 1 + 1, w = a + 1 + 1 | project x, z, z as y, w Fix #100174 Fix #100050 * Incorporate #100357 * Temporarily disable failing test Relates #100365 (cherry picked from commit 0ef4da2) * ESQL: Temporarily disable huge concat tests (#100352) We're working on these and we have a plan! (cherry picked from commit acca114) --------- Co-authored-by: Nik Everett <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Evals that introduce aliased can be simplified by extracting them into a project (and thus signaling there's no underlying processing).
The following eval:
eval x = a + 1, y = x, z = y + 1, y = z, w = y + 1can be converted into:
eval x = a + 1, z = a + 1 + 1, w = a + 1 + 1 | project x, z, z as y, wThis helps fuse the project with other projections/aggs downstream.
The text was updated successfully, but these errors were encountered: