Skip to content

fix(optimizer): do not hoist subqueries across anonymous function boundaries #8884

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

Merged
georgesittas merged 1 commit into from
Nov 4, 2026
+9 −1
Merged

fix(optimizer): do not hoist subqueries across anonymous function boundaries #8884

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sqlglot-integration-tests
Open in desktop
Submodule sqlglot-integration-tests updated from 97ce58 to 320f86
8 changes: 8 additions & 0 deletions sqlglot/optimizer/unnest_subqueries.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,14 @@ def unnest_subqueries(expression: E) -> E:
if not parent:
continue

# Don't hoist anonymous functions because their input-row versus post-aggregation role is unknown.
if isinstance(select.find_ancestor(exp.Anonymous, exp.Select), exp.Anonymous) or (
not _has_plain_explicit_group(parent)
and _is_post_aggregation(select, parent)
and any(find_in_scope(projection, exp.Anonymous) for projection in parent.selects)
):
continue

# Don't hoist a subquery out of a correlated, ungrouped aggregate if it's evaluated after the
# aggregation: the aggregate returns a row even for keys with no input rows, and the hoisted
# MAX(...) would turn the row's value into NULL
Expand Down
Loading