-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Labels
enhancementNew feature or requestNew feature or request
Description
Is your feature request related to a problem or challenge?
While looking at #18458, I realized we don't have ListView and LargeListView in ScalarValue.
There are some recent PRs supporting ListView and LargeListView (#18351, #18350).
I think it make sense to add them to ScalarValue.
Describe the solution you'd like
Add ListView and LargeListView to ScalarValue:
datafusion/datafusion/common/src/scalar/mod.rs
Lines 324 to 326 in 6ed22bb
| pub enum ScalarValue { | |
| /// represents `DataType::Null` (castable to/from any other type) | |
| Null, |
Describe alternatives you've considered
No response
Additional context
This might need to wait after arrow-rs 57.1.0 because there are some list view improvement recently.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request