Jefffrey commented on PR #17610: URL: https://github.com/apache/datafusion/pull/17610#issuecomment-3358843436
Yeah I believe `PropagateEmptyRelation` rule already covers this: https://github.com/apache/datafusion/blob/297e537bdfe0ce74e97cc8fd9875bd1af8f18a25/datafusion/optimizer/src/propagate_empty_relation.rs#L150-L190 Tests too: https://github.com/apache/datafusion/blob/297e537bdfe0ce74e97cc8fd9875bd1af8f18a25/datafusion/optimizer/src/propagate_empty_relation.rs#L323-L424 For the original issue (#17568), it spawned from a PR (#17449) which spawned from an issue (#17052) which encountered an error only because they manually constructed the `UnionExec` in the physical plan. I don't think there's anything to fix in that case, unless we do have a reproducing query. I'll close the issue #17568, thanks for checking into this @jonathanc-n -- 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
