alamb commented on issue #10426: URL: https://github.com/apache/datafusion/issues/10426#issuecomment-2110217302
> I have a WIP commit here: https://github.com/peter-toth/arrow-datafusion/commit/e8447996462ae4710b573c7088bc6d8b1e586cfb that adds TreeNode::apply_ref() / TreeNode::visit_ref(). > Using apply_ref() in the above example would make it work, but I haven't opened a PR yet as there are a few things to consider: > ... > But since I'm only aware of this ticket that requires this change to the APIs, I haven't opened the PR yet. Here is one example API that I would love to implement with such a tree-node api: https://github.com/apache/datafusion/issues/10505 I also ran into an example when trying to find embedded `Subquery`s in an `Expr` in https://github.com/apache/datafusion/blob/424757f365ea9eab7b242f5b604ef804baecd666/datafusion/optimizer/src/scalar_subquery_to_join.rs#L54-L68 -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
