Skip to content

Conversation

taniabogatsch
Copy link
Contributor

UNNEST is not a scalar function; thus, nesting it in scalar functions such as list_transform, etc., should not be supported.

@Mytherin, should we throw a Binder Error instead?

Fixes https://github.com/duckdblabs/duckdb-internal/issues/3059.

@duckdb-draftbot duckdb-draftbot marked this pull request as draft September 17, 2024 11:32
@taniabogatsch taniabogatsch marked this pull request as ready for review September 17, 2024 11:33
@Mytherin
Copy link
Collaborator

Thanks for the PR! LGTM. I think throwing directly here is fine.

@Mytherin Mytherin merged commit 04a1f75 into duckdb:main Sep 17, 2024
43 checks passed
@taniabogatsch taniabogatsch deleted the lambda-fix branch September 17, 2024 15:41
github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Sep 27, 2024
[Fix] Throw exception for UNNEST in lambdas (duckdb/duckdb#13969)
Fix partitions on wide tables (duckdb/duckdb#13988)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Sep 27, 2024
[Fix] Throw exception for UNNEST in lambdas (duckdb/duckdb#13969)
Fix partitions on wide tables (duckdb/duckdb#13988)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants