viirya commented on code in PR #7885:
URL: https://github.com/apache/arrow-datafusion/pull/7885#discussion_r1367273147
##########
datafusion/physical-expr/src/intervals/interval_aritmetic.rs:
##########
@@ -194,6 +194,23 @@ impl Display for IntervalBound {
/// called *unbounded* endpoint and represented using a `NULL` and written
using
/// `∞`.
///
+/// # Boolean Handling
+///
+/// Boolean values require special handling. Boolean Intervals NEVER have open
+/// bounds. If you try to and construct one one with open bounds,
+/// [`Interval::new`] will remap them to one of the three valid values.
+///
+/// Given there are only two boolean values, and they are ordered such that
Review Comment:
```suggestion
/// Given there are only two boolean values, and they are ordered such that
```
--
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]
