berkaysynnada commented on PR #15570: URL: https://github.com/apache/datafusion/pull/15570#issuecomment-2781422523
> I wonder if there is any way to write some tests for this (perhaps via `EXPLAIN` in .slt tests to demonstrate that the unecessary exec is removed) I don't think a redundant exec can be removed since if the input is 1 partition, it already does not introduce a new one, and what we see is input is always 1 (enforce distribution does its job 👏🏻), because we don't get any plan changes -- 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-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
