Deprecate ScalarUDF::is_nullable (#17074)

findepi · web-flow · commit 060938bf68a6 · 2025-08-07T22:57:10.000-07:00
`ScalarUDFImpl::is_nullable` is deprecated. The `ScalarUDF`'s wrapper
also needs deprecation to allow eventually removing the deprecated
`ScalarUDFImpl::is_nullable`.
diff --git a/datafusion/expr/src/udf.rs b/datafusion/expr/src/udf.rs
@@ -199,8 +199,9 @@ impl ScalarUDF {
         self.inner.simplify(args, info)
     }
 
-    #[allow(deprecated)]
+    #[deprecated(since = "50.0.0", note = "Use `return_field_from_args` instead.")]
     pub fn is_nullable(&self, args: &[Expr], schema: &dyn ExprSchema) -> bool {
+        #[allow(deprecated)]
         self.inner.is_nullable(args, schema)
     }
 

Original file line number	Diff line number	Diff line change
`@@ -199,8 +199,9 @@ impl ScalarUDF {`
`199`	`199`	`self.inner.simplify(args, info)`
`200`	`200`	`}`
`201`	`201`
`202`		`- #[allow(deprecated)]`
	`202`	+ #[deprecated(since = "50.0.0", note = "Use `return_field_from_args` instead.")]
`203`	`203`	`pub fn is_nullable(&self, args: &[Expr], schema: &dyn ExprSchema) -> bool {`
	`204`	`+ #[allow(deprecated)]`
`204`	`205`	`self.inner.is_nullable(args, schema)`
`205`	`206`	`}`
`206`	`207`