add missed out [#expect(deprecated)] assertions

Author: codetyri0n

datafusion/expr/src/udaf.rs

@@ -208,8 +208,12 @@ impl AggregateUDF {
         self.inner.window_function_display_name(params)
     }
 
+    #[deprecated(
+        since = "52.0.0",
+        note = "Use `return_field` instead with return_field.is_nullable()."
+    )]
     pub fn is_nullable(&self) -> bool {
-        #[allow(deprecated)]
+        #[expect(deprecated)]
         self.inner.is_nullable()
     }
 
@@ -534,7 +538,7 @@ pub trait AggregateUDFImpl: Debug + DynEq + DynHash + Send + Sync {
     /// but others like `MIN` will return `NULL` if there is nullable input.
     /// Note that if the function is declared as *not* nullable, make sure the [`AggregateUDFImpl::default_value`] is `non-null`
     #[deprecated(
-        since = "51.0.0",
+        since = "52.0.0",
         note = "Use `return_field` instead with return_field.is_nullable()."
     )]
     fn is_nullable(&self) -> bool {
@@ -1105,7 +1109,7 @@ pub fn udaf_default_return_field<F: AggregateUDFImpl + ?Sized>(
     Ok(Arc::new(Field::new(
         func.name(),
         data_type,
-        #[allow(deprecated)]
+        #[expect(deprecated)]
         {
             func.is_nullable()
         },
@@ -1255,7 +1259,7 @@ impl AggregateUDFImpl for AliasedAggregateUDFImpl {
     }
 
     fn is_nullable(&self) -> bool {
-        #[allow(deprecated)]
+        #[expect(deprecated)]
         self.inner.is_nullable()
     }
 

datafusion/ffi/src/udaf/mod.rs

@@ -344,6 +344,7 @@ impl From<Arc<AggregateUDF>> for FFI_AggregateUDF {
     fn from(udaf: Arc<AggregateUDF>) -> Self {
         let name = udaf.name().into();
         let aliases = udaf.aliases().iter().map(|a| a.to_owned().into()).collect();
+        #[expect(deprecated)]
         let is_nullable = udaf.is_nullable();
         let volatility = udaf.signature().volatility.into();