alamb commented on PR #19538: URL: https://github.com/apache/datafusion/pull/19538#issuecomment-3811041766
> I've gotten the logical optimizer side of this working in #20036. It's not too bad, +700 LOC. But this is already a big PR and then we can look at the full picture and e.g. see if we can simplify the physical optimizer side. Wdyt of merging that into here @alamb ? I am not quite sure what you are proposing? You mean merge #20036 into this PR? #20036 seems to have more than 700 LOC <img width="234" height="63" alt="Screenshot 2026-01-28 at 7 28 29 AM" src="https://github.com/user-attachments/assets/07501426-72ae-4840-869b-d8ed35c10f34"; /> I am sorry but I have a really hard time finding the bandwidth to review such large PRs in depth. I am really struggling (across all the arrow/datafusion projects) -- 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]
