datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard
verified publisher icon apache

[Epic] Improved TreeNode APIs

Open alamb opened this issue 1 year ago • 2 comments

Is your feature request related to a problem or challenge?

This is a follow on to https://github.com/apache/arrow-datafusion/issues/8913 that tracks additional improvements to the TreeNode API

Describe the solution you'd like

  • [x] https://github.com/apache/arrow-datafusion/issues/10097

alamb avatar Apr 17 '24 17:04 alamb

The ConcreteTreeNode seems like quite a nice abstraction. Maybe it's interesting to add for the LogicalPlan & Expr as well.

backkem avatar May 08 '24 11:05 backkem

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

alamb avatar May 08 '24 13:05 alamb