fix clippy

Author: rkrishn7

datafusion/functions/src/core/named_struct.rs

@@ -185,7 +185,7 @@ impl ScalarUDFImpl for NamedStructFunc {
         &self,
         literal_args: &[Option<ScalarValue>],
     ) -> Option<StructFieldMapping> {
-        if literal_args.is_empty() || literal_args.len() % 2 != 0 {
+        if literal_args.is_empty() || !literal_args.len().is_multiple_of(2) {
             return None;
         }
 

datafusion/physical-expr/src/projection.rs

@@ -641,7 +641,7 @@ impl ProjectionMapping {
                 None => Ok(Transformed::no(e)),
             })
             .data()?;
-            map.entry(source_expr.clone())
+            map.entry(Arc::clone(&source_expr))
                 .or_default()
                 .push((Arc::clone(&target_expr), expr_idx));
 
@@ -670,7 +670,8 @@ impl ProjectionMapping {
                 {
                     if let DataType::Struct(struct_fields) = func_expr.return_type() {
                         for (accessor_args, source_arg_idx) in &field_mapping.fields {
-                            let value_expr = func_expr.args()[*source_arg_idx].clone();
+                            let value_expr =
+                                Arc::clone(&func_expr.args()[*source_arg_idx]);
 
                             // Build accessor args: [target_col, ...field_name_literals]
                             let mut accessor_fn_args: Vec<Arc<dyn PhysicalExpr>> =