andygrove commented on code in PR #14279:
URL: https://github.com/apache/datafusion/pull/14279#discussion_r1937577475
##########
datafusion/physical-expr/src/analysis.rs:
##########
@@ -81,8 +81,12 @@ impl AnalysisContext {
#[derive(Clone, Debug, PartialEq)]
pub struct ExprBoundaries {
pub column: Column,
- /// Minimum and maximum values this expression can have.
- pub interval: Interval,
+ /// Minimum and maximum values this expression can have. A `None` value
+ /// indicates that evaluating the given column results in an empty set.
+ /// For example, if the column `a` has values in the range [10, 20],
+ /// and there is a filter asserting that `a > 50`, then the resulting
interval
+ /// range of `a` will be `None`.
+ pub interval: Option<Interval>,
Review Comment:
I added the `api-change` label due to making `interval` an `Option`.
--
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]
