alamb commented on issue #1228:
URL: 
https://github.com/apache/arrow-datafusion/issues/1228#issuecomment-970187248


   > above all, I think this is a meaningful and valuable refactoring, despite 
major changes to the codebase.
   
   Yes, I think this is the primary benefit of pulling LogicalPlan nodes into 
their own structs  (not that it is required for subqueries -- I think we could 
probably make that work somehow -- but that this will make the codebase cleaner 
in the longer term


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