xudong963 commented on PR #21107: URL: https://github.com/apache/datafusion/pull/21107#issuecomment-4125197296
@alamb thanks for the review, this PR is not trading parallelism for sortedness, and it does not remove the hash repartition. The plan still uses the same Hash repartition with the same partition count; the only change is that we keep the order-preserving variant when removing it would cause the parent AggregateExec (in Sorted / PartiallySorted mode) to switch from incremental to blocking execution. -- 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]
