revisit union interval logic

apache · Feb 14, 2025 · aaf116b · aaf116b
1 parent b4bd851
commit aaf116b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/datafusion/expr-common/src/interval_arithmetic.rs b/datafusion/expr-common/src/interval_arithmetic.rs
@@ -646,8 +646,9 @@ impl Interval {
         };
 
         // If the intervals overlap or touch, return a single merged interval
-        if (self.is_not_null() && rhs.is_not_null()) && self.upper >= rhs.lower
-            || rhs.upper >= self.lower
+        if self.is_not_null()
+            && rhs.is_not_null()
+            && (self.upper >= rhs.lower || rhs.upper >= self.lower)
         {
             return Ok(Some(Self::new(lower_bound.unwrap(), upper_bound.unwrap())));
         }