jackwener commented on PR #4783: URL: https://github.com/apache/arrow-datafusion/pull/4783#issuecomment-1368454405
> Thank you @jackwener -- this looks great. 🙏 the planner is in desperate need of some cleaning up > > Something else that would help I think is to move the tests from `datafusion/sql/src/planner.rs` into `datafusion/sql/test/planner.rs` or similar to help parallelize the test compilation > > Sadly I have just created a bunch of conflicts in in #4779 (also related to shrinking the planner) > > Also related (and conflicting): #4785 #4787 I make this PR draft, I will wait for those PR merge. Thanks @alamb remind -- 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]
