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

   ## Which issue does this PR close?
   
   Related to XXX
   
   Running benchmarks to see if this helps
   
   ## Rationale for this change
   
   I spent time profiling the sql_planner benchmarks as part of 
https://github.com/apache/datafusion/pull/17419
   and the amount of time spent planning unions was surprising.
   
   
   
   ## What changes are included in this PR?
   
   <!--
   There is no need to duplicate the description in the issue here but it is 
sometimes worth providing a summary of the individual changes in this PR.
   -->
   
   ## Are these changes tested?
   
   I also felt bad we haven't really improved planning speed in a while!
   
   I couldn't help myself and implemented a simple optimization to flatten 
unions
   as soon as possible in the planner rather than waiting maybe for the 
optimizers to do it.
   The basic idea is quite simple: if the input is a union, just add its inputs 
to the list of inputs
   
   ## Are there any user-facing changes?
   
   not sure
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to