adriangb commented on PR #15566: URL: https://github.com/apache/datafusion/pull/15566#issuecomment-2790778558
Some implementations do recurse I think, for similar reasons to our recursion here: https://github.com/pydantic/datafusion/blob/f8a6384bdf21b2eeb7bcfe3f08e52712735bb285/datafusion/physical-plan/src/projection.rs#L534-L535 It was @alamb that suggested we do it this way, unless I misunderstood his suggestion. I think it's possible to do the recursion as an optimizer rule but making the APIs flexible enough to handle all of the cases with joins, etc. gets very complicated especially because the recursion needs to probe and jump, transmit information back up, etc. -- 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
