notfilippo commented on PR #11978:
URL: https://github.com/apache/datafusion/pull/11978#issuecomment-2294834051
> 🤔 so that leaves us with the question of how to incrementally try and
introduce logical types into logical plans 🤔
As @jayzhan211 says, the type information is still retained in the Schema,
but it would be impractical to modify all expression orders to support the fact
that the type is not stored in the ColumnarValue.
My discussion around Datum revolves around the fact that to retain physical
type information maybe `ColumnarValue` enum should instead be:
```rust
enum ColumnarValue {
Array(Arc<dyn Array>),
Scalar(Arc<dyn Array>) // this is an array of len = 1
}
```
or that ScalarValue is kept as is and a LogicalScalarValue is introduced.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
