More clippy warnings

Author: timsaucer

src/dataframe.rs

@@ -470,7 +470,7 @@ impl PyDataFrame {
         self.df
             .as_ref()
             .parse_sql_expr(&expr)
-            .map(|e| PyExpr::from(e))
+            .map(PyExpr::from)
             .map_err(PyDataFusionError::from)
     }
 

src/expr/statement.rs

@@ -343,7 +343,7 @@ impl PyPrepare {
         let input = input.plan().clone();
         let fields = data_types
             .into_iter()
-            .map(|data_type| DataType::from(data_type))
+            .map(DataType::from)
             .enumerate()
             .map(|(idx, data_type)| Field::new(format!("field_{idx}"), data_type, true))
             .map(Arc::new)

src/udf.rs

@@ -37,10 +37,11 @@ use crate::utils::{parse_volatility, validate_pycapsule};
 
 /// Create a Rust callable function from a python function that expects pyarrow arrays
 fn pyarrow_function_to_rust(
-    func: Bound<PyAny>,
+    func: Py<PyAny>,
 ) -> impl Fn(&[ArrayRef]) -> Result<ArrayRef, DataFusionError> {
     move |args: &[ArrayRef]| -> Result<ArrayRef, DataFusionError> {
         Python::attach(|py| {
+            let func = func.bind(py);
             // 1. cast args to Pyarrow arrays
             let py_args = args
                 .iter()
@@ -68,7 +69,7 @@ fn pyarrow_function_to_rust(
 /// Create a DataFusion's UDF implementation from a python function
 /// that expects pyarrow arrays. This is more efficient as it performs
 /// a zero-copy of the contents.
-fn to_scalar_function_impl(func: Bound<PyAny>) -> ScalarFunctionImplementation {
+fn to_scalar_function_impl(func: Py<PyAny>) -> ScalarFunctionImplementation {
     // Make the python function callable from rust
     let pyarrow_func = pyarrow_function_to_rust(func);
 
@@ -93,7 +94,7 @@ impl PyScalarUDF {
     #[pyo3(signature=(name, func, input_types, return_type, volatility))]
     fn new(
         name: &str,
-        func: Bound<PyAny>,
+        func: Py<PyAny>,
         input_types: PyArrowType<Vec<DataType>>,
         return_type: PyArrowType<DataType>,
         volatility: &str,