ozankabak commented on PR #7942:
URL: 
https://github.com/apache/arrow-datafusion/pull/7942#issuecomment-1868282579

   I like this general effort and we will be happy to help. The main challenge 
I see is that this touches many files and procedures, and we may lose/break 
certain behaviors that are not adequately tested. Therefore, IMO it makes sense 
to first clean-up some of the tree traversal logic in our planner/optimization 
rules as a stepping stone to this.
   
   We will submit a cleanup PR early next week to simplify around half of the 
usages in physical planning/optimization so the job here will be easier.


-- 
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]

Reply via email to