alamb commented on issue #8078: URL: https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1800094571
Another challenge I found with using `Interval` as proposed in https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1798813378 is more mechanical but real: `Interval` is in the `datafusion_physical_expr` crate but the `Precision` are in datafusion_common, meaning I can't use `Interval` in `Precision` without moving code around. Not impossible to do, but a data point. I will try prototyping option 1 sugested by @berkaysynnada https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1798813378 and see what I find -- 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]
