jayzhan211 commented on issue #12622: URL: https://github.com/apache/datafusion/issues/12622#issuecomment-2639134313
Having a true "logical" Scalar sounds like a good idea, since either the current `ScalarValue` or `Scalar` are actually still tightly coupled with `DataType`, if we can bring `LogicalScalar` into LogicalPlan and transform to `ScalarValue` only if we need it, maybe we could reduce the usage of `ScalarValue` we have now. -- 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]
