alamb commented on issue #19849: URL: https://github.com/apache/datafusion/issues/19849#issuecomment-3761014192
> 🤔 So the issue is more about the partial AggregateExec declaring a technically incorrect output partitioning? I am just trying to understand the issue -- is this a correct summary? * partial `AggregateExec` declared an incorrect output partitioning (not new) * prior to https://github.com/apache/datafusion/pull/19304, this bug was hidden because the plans always repartitioned the output anyways. * After https://github.com/apache/datafusion/pull/19304, the bug caused incorrect answers because a RepartitioningExec was (incorrectly) removed based on the incorrect output partitioning -- 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]
