if expr in GetIndexedField is null, then always return nullable.

ArroyoSystems · Apr 18, 2024 · dcb1631 · dcb1631
1 parent 4b9d841
commit dcb1631
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/datafusion/expr/src/expr_schema.rs b/datafusion/expr/src/expr_schema.rs
@@ -316,10 +316,8 @@ impl ExprSchemable for Expr {
             Expr::GetIndexedField(GetIndexedField { expr, field }) => {
                 // If schema is nested, check if parent is nullable
                 // if it is, return early
-                if let Expr::Column(col) = expr.as_ref() {
-                    if input_schema.nullable(col)? {
-                        return Ok(true);
-                    }
+                if expr.nullable(input_schema)? {
+                    return Ok(true);
                 }
                 field_for_index(expr, field, input_schema).map(|x| x.is_nullable())
             }