notfilippo commented on PR #12536: URL: https://github.com/apache/datafusion/pull/12536#issuecomment-2370648121
> So one way to proceed if we are agreed on the basic idea is to create a new "Epic" ticket that lists the steps planned (it is fine if we don't know them all now). Here is an example [apache/arrow-rs#5374](https://github.com/apache/arrow-rs/issues/5374) I love the idea! My starting suggestion for potential issues is: - Add `Scalar` - Use `Scalar` in `Expr::Literal` - Introduce `LogicalType` enum and the _logically equivalent_ notion - Remove `ScalarValue::LargeUtf8` and `ScalarValue::Utf8View` in favour of `ScalarValue::Utf8` - Remove `ScalarValue::LargeBinary` and `ScalarValue::BinaryView` in favour of `ScalarValue::Utf8` - Remove `ScalarValue::Dictionary` (from #12488) -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
