hiltontj commented on issue #14309:
URL: https://github.com/apache/datafusion/issues/14309#issuecomment-2614450399
Another thing worth noting is that from what I've seen, DataFusion splits
filter expressions on `AND` conjunctions. So in practice, _I think_ this is
less of an issue, because the expression,
```
a < 10 AND b > "aaa"
```
would be split into two `Expr`s when pushed down, e.g.,
```rust
// this:
assert_eq!(
filters,
&[col("a").lt(10), col("b").gt(lit("aaa"))]
);
// NOT this:
assert_eq!(
filters,
&[col("a").lt(10).and(col("b").gt(lit("aaa")))]
);
```
And because analysis is done _per_ `Expr` in the list, then we wouldn't run
into the problem.
--
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
