(datafusion-comet) branch main updated: fix: if expr nullable (#2217)

[agrove]

This is an automated email from the ASF dual-hosted git repository.

agrove pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git


The following commit(s) were added to refs/heads/main by this push:
     new 0a0b65b13 fix: if expr nullable (#2217)
0a0b65b13 is described below

commit 0a0b65b139b74b0d0d8d0717d646a2beaa119265
Author: Asura7969 <1402357...@qq.com>
AuthorDate: Sat Aug 23 01:37:49 2025 +0800

    fix: if expr nullable (#2217)
---
 native/spark-expr/src/conditional_funcs/if_expr.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/native/spark-expr/src/conditional_funcs/if_expr.rs 
b/native/spark-expr/src/conditional_funcs/if_expr.rs
index 58adaea17..8481e7e87 100644
--- a/native/spark-expr/src/conditional_funcs/if_expr.rs
+++ b/native/spark-expr/src/conditional_funcs/if_expr.rs
@@ -98,7 +98,7 @@ impl PhysicalExpr for IfExpr {
     }
 
     fn nullable(&self, _input_schema: &Schema) -> Result<bool> {
-        if self.true_expr.nullable(_input_schema)? || 
self.true_expr.nullable(_input_schema)? {
+        if self.true_expr.nullable(_input_schema)? || 
self.false_expr.nullable(_input_schema)? {
             Ok(true)
         } else {
             Ok(false)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org