Add custom nullability to SparkMakeInterval

- Implement return_field_from_args for SparkMakeInterval
- With no arguments: returns non-nullable (always zero interval)
- With arguments: returns nullable (input nulls or overflow possible)

Closes #19155

Author: bhanureddy-int

datafusion/spark/src/function/datetime/make_interval.rs

@@ -21,12 +21,12 @@ use std::sync::Arc;
 use arrow::array::{Array, ArrayRef, IntervalMonthDayNanoBuilder, PrimitiveArray};
 use arrow::datatypes::DataType::Interval;
 use arrow::datatypes::IntervalUnit::MonthDayNano;
-use arrow::datatypes::{DataType, IntervalMonthDayNano};
+use arrow::datatypes::{DataType, Field, FieldRef, IntervalMonthDayNano};
 use datafusion_common::types::{NativeType, logical_float64, logical_int32};
 use datafusion_common::{DataFusionError, Result, ScalarValue, plan_datafusion_err};
 use datafusion_expr::{
-    Coercion, ColumnarValue, ScalarFunctionArgs, ScalarUDFImpl, Signature, TypeSignature,
-    TypeSignatureClass, Volatility,
+    Coercion, ColumnarValue, ReturnFieldArgs, ScalarFunctionArgs, ScalarUDFImpl, Signature,
+    TypeSignature, TypeSignatureClass, Volatility,
 };
 use datafusion_functions::utils::make_scalar_function;
 
@@ -122,6 +122,31 @@ impl ScalarUDFImpl for SparkMakeInterval {
         Ok(Interval(MonthDayNano))
     }
 
+    fn return_field_from_args(&self, args: ReturnFieldArgs<'_>) -> Result<FieldRef> {
+        // Spark make_interval semantics:
+        // - Returns NULL if ANY input argument is NULL
+        // - Returns NULL on arithmetic overflow (e.g., year * 12 + month overflows)
+        // - With no arguments, returns a non-null zero interval
+        //
+        // Therefore:
+        // - If no arguments: not nullable (always returns 0 interval)
+        // - If any arguments: nullable (inputs might be null OR overflow might occur)
+        let nullable = if args.arg_fields.is_empty() {
+            false
+        } else {
+            // Always nullable when we have arguments because:
+            // 1. Any nullable input → nullable output
+            // 2. Overflow can occur even with non-null inputs → nullable output
+            true
+        };
+
+        Ok(Arc::new(Field::new(
+            "make_interval",
+            Interval(MonthDayNano),
+            nullable,
+        )))
+    }
+
     fn invoke_with_args(&self, args: ScalarFunctionArgs) -> Result<ColumnarValue> {
         if args.args.is_empty() {
             return Ok(ColumnarValue::Scalar(ScalarValue::IntervalMonthDayNano(