alamb commented on issue #10121: URL: https://github.com/apache/datafusion/issues/10121#issuecomment-2100549720
> The `ConcreteTreeNode` seems like quite a nice abstraction. Maybe it's interesting to add for the `LogicalPlan` & `Expr` as well. Sounds like a nice idea to me -- I suspect the best way to do so would be to find a usecase that needed / could benefit from the use of that API and refactor the existing code into the (new) API -- 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
