Rachelint commented on issue #12596: URL: https://github.com/apache/datafusion/issues/12596#issuecomment-4793274934
Prove to be not promising through #23149 The reason is: - `RepartitionExec` has been highly optimized now, and usually occupy 5-% cpu time in query - If we want to support the fusing approach with low code changes like #23149 , the performance may even worse than current - If we pay much effort to make the large refactor, just possible to get 5-% faster, it is not really valuable for me -- 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]
