Skip to content

feat: Add support for null literal with struct type#797

Merged
andygrove merged 1 commit intoapache:mainfrom
eejbyfeldt:i796-null-struct-literal
Aug 9, 2024
Merged

feat: Add support for null literal with struct type#797
andygrove merged 1 commit intoapache:mainfrom
eejbyfeldt:i796-null-struct-literal

Conversation

@eejbyfeldt
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #796.

Rationale for this change

Support null literals in more cases. Useful for creating test cases in #735 but probably will come in useful in more cases in the future.

What changes are included in this PR?

Add support for null literals with struct data type.

How are these changes tested?

New test case.

@eejbyfeldt eejbyfeldt changed the title Add support for null literal with struct type feat: Add support for null literal with struct type Aug 8, 2024
@eejbyfeldt eejbyfeldt marked this pull request as ready for review August 8, 2024 18:30
Copy link
Copy Markdown
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

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

Thasnks @eejbyfeldt

@andygrove andygrove merged commit 1374f2a into apache:main Aug 9, 2024
himadripal pushed a commit to himadripal/datafusion-comet that referenced this pull request Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for null with struct data type

4 participants