alamb commented on issue #7882: URL: https://github.com/apache/arrow-datafusion/issues/7882#issuecomment-1773016192
Thanks @berkaysynnada -- note I just proposed three relatively small changes . Perhaps you have some time to look at them: * https://github.com/apache/arrow-datafusion/pull/7885 * https://github.com/apache/arrow-datafusion/pull/7884 * https://github.com/apache/arrow-datafusion/pull/7886 > I also plan to move the code into a more accessible place (possibly datafusion_expr) What would you think about making a `datafusion-interval` crate to hold the interval analysis library ? that would keep it nicely isolated and likely improve the development cycle time (I found it quite painful to work on in datafusion-physical-expr) -- 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]
