berkaysynnada commented on issue #8078: URL: https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1801757513
> Another challenge I found with using `Interval` as proposed in option 2 of [#8078 (comment)](https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1798813378) is more mechanical but real: `Interval` is in the `datafusion_physical_expr` crate but `Precision` is 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 [#8078 (comment)](https://github.com/apache/arrow-datafusion/issues/8078#issuecomment-1798813378) and see what I find Just so you know, I have completed the interval refactor and submitted it for our internal review. After that, interval arithmetic will be in `datafusion_expr`, and there will be no significant obstacles to moving it to `datafusion_common`. -- 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]
