alamb commented on issue #7142:
URL:
https://github.com/apache/arrow-datafusion/issues/7142#issuecomment-1658164958
To be clear, to be consistent with the rest of the datafusion type system, I
think we should stop using DataType::Null to represent null constants in array
functions and instead move to using `DataType::XXX` where XXX is the type of
the other list elements.
So after type coercion I would expect `make_list(1, 2, NULL)` to be
```
make_list(
ScalarValue::Int64(Some(1)),
ScalarValue::Int64(Some(2)),
ScalarValue::Int64(None), // NOTE this is not ScalarValue::Null
) -> ListArray w/ Int64 value type
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
