build(deps): update versions

Author: dariocurr

Cargo.toml

@@ -31,17 +31,17 @@ paste = "1.0"
 
 [dependencies.datafusion]
 default-features = false
-version = "48.0"
+version = "49.0"
 
 [dev-dependencies]
-criterion = "0.6"
-rand = "0.8"
-tokio = "1.46"
+criterion = "0.7"
+rand = "0.9"
+tokio = "1.47"
 
 [dev-dependencies.arrow]
 default-features = false
 features = ["test_utils"]
-version = "55.0"
+version = "56.0"
 
 [dev-dependencies.insta]
 features = ["yaml"]

src/macros.rs

@@ -28,7 +28,7 @@ macro_rules! make_udaf_expr {
                 vec![$($arg),*],
                 false,
                 None,
-                None,
+                vec![],
                 None,
             ))
         }

src/max_min_by.rs

@@ -80,7 +80,7 @@ impl logical_expr::AggregateUDFImpl for MaxByFunction {
     fn simplify(&self) -> Option<logical_expr::function::AggregateFunctionSimplification> {
         let simplify = |mut aggr_func: logical_expr::expr::AggregateFunction,
                         _: &dyn logical_expr::simplify::SimplifyInfo| {
-            let mut order_by = aggr_func.params.order_by.unwrap_or_default();
+            let mut order_by = aggr_func.params.order_by;
             let (second_arg, first_arg) = (aggr_func.params.args.remove(1), aggr_func.params.args.remove(0));
             let sort = logical_expr::expr::Sort::new(second_arg, true, false);
             order_by.push(sort);
@@ -89,7 +89,7 @@ impl logical_expr::AggregateUDFImpl for MaxByFunction {
                 vec![first_arg],
                 aggr_func.params.distinct,
                 aggr_func.params.filter,
-                Some(order_by),
+                order_by,
                 aggr_func.params.null_treatment,
             ));
             Ok(func)
@@ -165,7 +165,7 @@ impl logical_expr::AggregateUDFImpl for MinByFunction {
     fn simplify(&self) -> Option<logical_expr::function::AggregateFunctionSimplification> {
         let simplify = |mut aggr_func: logical_expr::expr::AggregateFunction,
                         _: &dyn logical_expr::simplify::SimplifyInfo| {
-            let mut order_by = aggr_func.params.order_by.unwrap_or_default();
+            let mut order_by = aggr_func.params.order_by;
             let (second_arg, first_arg) = (aggr_func.params.args.remove(1), aggr_func.params.args.remove(0));
 
             let sort = logical_expr::expr::Sort::new(second_arg, false, false);
@@ -175,7 +175,7 @@ impl logical_expr::AggregateUDFImpl for MinByFunction {
                 vec![first_arg],
                 aggr_func.params.distinct,
                 aggr_func.params.filter,
-                Some(order_by),
+                order_by,
                 aggr_func.params.null_treatment,
             ));
             Ok(func)