alamb opened a new pull request, #4907:
URL: https://github.com/apache/arrow-datafusion/pull/4907

   Stop copying Exprs so much in logical planning
   
   Draft as it builds off
   https://github.com/apache/arrow-datafusion/pull/4906/
   
   `LogicalPlan::expressions()` actually copies the expressions. Many times 
these expressions are then simply passed to `new_from_plan` and then copied 
*again*
   
   Using the function added to 
https://github.com/apache/arrow-datafusion/pull/4906/ this PR avoids copying 
all plan expressions in several places
   
   
   I ran out of time, but this direction seems promising
   


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