fzlzjerry commented on PR #24258: URL: https://github.com/apache/datafusion/pull/24258#issuecomment-5265976072
Addressed both plan concerns in c2bdac8 and synced the branch with the latest `main`. The general expression simplification still preserves NULL in projection contexts, while `EliminateFilter` now reduces the generated positive `AND` / `OR` trees using filter truth semantics. As a result, the disjoint `IN` case is back to `EmptyRelation`, and the `NOT IN` / `OR` case becomes `IS NOT NULL` rather than retaining the compound predicate. The latter must not become a bare scan because NULL inputs do not satisfy the filter. The overlapping `log`, `power`, and XOR changes are no longer part of this PR. The remaining 8-file scope passes all-target/all-feature clippy, `./dev/rust_lint.sh`, the SQLLogicTest suite, and the extended workspace test matrix. -- 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]
