alamb commented on PR #9913:
URL: 
https://github.com/apache/arrow-datafusion/pull/9913#issuecomment-2042283375

   > I was thinking about this a bit more and I unfortunately think this PR is 
still an API change.
   
   That makes sense -- I have added the label back.
   
   > But I have no other idea to fix the current inconsistency between 
   
   I think it is inevitable that there are different calls for `LogicalPlan` 
for traversing with subqueries and without subqueries
   
   
   > I agree. I'm happy to take that task,
   
   Awesome -- thank you. I filed 
https://github.com/apache/arrow-datafusion/issues/9994 to track the idea. I may 
make a few small PRs in this direction to start the process too
   
   
   


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