[GitHub] [arrow-datafusion] metesynnada opened a new issue, #5878: Moving PipelineFixer above all rules to use ExecutionPlan APIs

["metesynnada (via GitHub)" <gi...@apache.org>]

metesynnada opened a new issue, #5878:
URL: https://github.com/apache/arrow-datafusion/issues/5878

   ### Is your feature request related to a problem or challenge?
   
   `PipelineFixer` may change the original plan in order to fit unbounded sources into the query. However, currently, the swap is done below a distribution rule (`Repartition`), which uses an unchanged plan to add `RepartitionExec::RoundRobin`. 
   
   `PipelineFixer` (maybe more rules in the future) can change the `ExecutionPlan` at any level, and the new `ExecutionPlan` can have a different set of flags (maybe ordering, distribution, or more). 
   
   ### Describe the solution you'd like
   
   I suggest making the executor changer rules above the rules that fill the sort, distribution, etc.
   
   ### Describe alternatives you've considered
   
   NA
   
   ### Additional context
   
   NA


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] alamb closed issue #5878: Moving PipelineFixer above all rules to use ExecutionPlan APIs

["alamb (via GitHub)" <gi...@apache.org>]

alamb closed issue #5878: Moving PipelineFixer above all rules to use ExecutionPlan APIs
URL: https://github.com/apache/arrow-datafusion/issues/5878


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org