alamb commented on PR #9913: URL: https://github.com/apache/arrow-datafusion/pull/9913#issuecomment-2040625078
> @alamb, I've updated the PR. I will try to add tests for the `LogicalPlan::..._with_subqueries()` APIs in this PR or a in follow-up PR on Monday. I think it would be great to do this as a follow up PR on Monday, however I think the utility of unit tests on this code is somewhat limited. It will be tested end to end once I switch the optimizers to using the TreeNode API in https://github.com/apache/arrow-datafusion/pull/9954. I took the liberty of pushing a commit to this branch to fix clippy and plan to merge it in when CI passes -- 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]
