Nested `unnest` of a struct reports an internal error ("Assertion failed: struct_allowed") instead of a planning error

Open Beginner friendly
#24,911 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
85/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
rust
Domain
databases

Research direction

Start in datafusion/sql/src/utils.rs at RecursiveUnnestRewriter::transform, where struct_allowed currently produces the internal error. Review the unnest.slt coverage, including the arrow_typeof(unnest(column5)) case, and run the relevant SQL tests. Done means nested struct unnest expressions report the stated planning error instead of an internal error.

Written by the indexing model from the issue text.

Description

Describe the bug

unnest on a struct is only supported as the top-level select expression. Using it anywhere else hits assert_or_internal_err!(struct_allowed, ...) in RecursiveUnnestRewriter::transform (datafusion/sql/src/utils.rs), so the user gets an internal error that asks them to file a bug, even though the condition is entirely determined by the shape of their query.

To Reproduce
SELECT unnest(unnest(struct([1])));
SELECT unnest(struct(1))['c0'];
Internal error: Assertion failed: struct_allowed: unnest on struct can only be applied at the root level of select expression.
This issue was likely caused by a bug in DataFusion's code. Please help us to resolve this by filing a bug report in our issue tracker: https://github.com/apache/datafusion/issues

unnest.slt even has a test (select arrow_typeof(unnest(column5)) from unnest_table) that pins this internal-error text.

Expected behavior
Error during planning: unnest on struct can only be applied at the root level of select expression
Additional context

Found while running a corpus of unusual unnest expressions against datafusion-cli.

Dominant language
Rust
Stars
9.3k
Forks
2.4k
Avg merge
3d 8h
Merged PRs (30d)
354

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from apache/datafusion

All issues in apache/datafusion

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.