Jimexist commented on issue #1179:
URL:
https://github.com/apache/arrow-datafusion/issues/1179#issuecomment-953082109
> The source of the problem is here:
>
> ```
> SQLExpr::Value(Value::Null) => {
> Ok(Expr::Literal(ScalarValue::Utf8(None)))
> }
> ```
>
> I would propose:
>
> * To introduce `ScalarValue::Null` and use that instead for `Value::Null`.
> * Return `DataType::Null` as type for `ScalarValue::Null`
> * Implement the rest of changes in coercion as needed.
i wonder how this line up with typed nulls for other types.
or, could we actually get rid of the `Option` wrapping for other types, and
keep this null enum case with wrapper type instead?
--
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]
