yashaggarwal85d commented on issue #26622: URL: https://github.com/apache/datafusion/issues/26622#issuecomment-6357636601
Hi @alpaylan, thanks for the detailed report and the pointer to `NamePreserver`. I saw you mentioned wanting to fix this yourself. If you haven't started yet, I have a small fix ready that wraps the rewrite in `RewriteSetComparison::rewrite_plan` with `NamePreserver`, plus sqllogictest cases for `> ALL`, `= ANY`, and `< ANY` in the SELECT list. Would you mind if I open the PR? I'd leave the `bool_or(...)` variant (the "Physical plan does not support logical expression Exists" limitation) out of scope so it can be tracked separately, and I'm happy to step back if you already have something in progress. -- 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]
