minor: fix clippy problem in new version.

datafusion/common/src/dfschema.rs

@@ -519,7 +519,7 @@ impl From<DFSchema> for SchemaRef {
 }
 
 // Hashing refers to a subset of fields considered in PartialEq.
-#[allow(clippy::derive_hash_xor_eq)]
+#[allow(clippy::derived_hash_with_manual_eq)]
 impl Hash for DFSchema {
     fn hash<H: std::hash::Hasher>(&self, state: &mut H) {
         self.fields.hash(state);

datafusion/core/src/physical_plan/aggregates/no_grouping.rs

@@ -164,7 +164,7 @@ impl RecordBatchStream for AggregateStream {
 
 /// Perform group-by aggregation for the given [`RecordBatch`].
 ///
-/// If successfull, this returns the additional number of bytes that were allocated during this process.
+/// If successful, this returns the additional number of bytes that were allocated during this process.
 ///
 /// TODO: Make this a member function
 fn aggregate_batch(

datafusion/core/src/physical_plan/aggregates/row_hash.rs

@@ -297,7 +297,7 @@ impl RecordBatchStream for GroupedHashAggregateStream {
 impl GroupedHashAggregateStream {
     /// Perform group-by aggregation for the given [`RecordBatch`].
     ///
-    /// If successfull, this returns the additional number of bytes that were allocated during this process.
+    /// If successful, this returns the additional number of bytes that were allocated during this process.
     ///
     fn group_aggregate_batch(&mut self, batch: RecordBatch) -> Result<usize> {
         // Evaluate the grouping expressions:

datafusion/expr/src/logical_plan/plan.rs

@@ -1620,7 +1620,7 @@ pub struct CreateExternalTable {
 }
 
 // Hashing refers to a subset of fields considered in PartialEq.
-#[allow(clippy::derive_hash_xor_eq)]
+#[allow(clippy::derived_hash_with_manual_eq)]
 impl Hash for CreateExternalTable {
     fn hash<H: Hasher>(&self, state: &mut H) {
         self.schema.hash(state);
@@ -1719,7 +1719,7 @@ pub struct Analyze {
 }
 
 /// Extension operator defined outside of DataFusion
-#[allow(clippy::derive_hash_xor_eq)] // see impl PartialEq for explanation
+#[allow(clippy::derived_hash_with_manual_eq)] // see impl PartialEq for explanation
 #[derive(Clone, Eq, Hash)]
 pub struct Extension {
     /// The runtime extension operator

datafusion/physical-expr/src/aggregate/count_distinct.rs

@@ -112,7 +112,7 @@ impl DistinctCountAccumulator {
                 .values
                 .iter()
                 .next()
-                .map(|vals| ScalarValue::size(vals) - std::mem::size_of_val(&vals))
+                .map(|vals| ScalarValue::size(vals) - std::mem::size_of_val(vals))
                 .unwrap_or(0)
     }
 }

datafusion/sql/src/expr/arrow_cast.rs

@@ -127,10 +127,10 @@ impl<'a> Parser<'a> {
         let data_type = self.parse_next_type()?;
         // ensure that there is no trailing content
         if self.tokenizer.next().is_some() {
-            return Err(make_error(
+            Err(make_error(
                 self.val,
                 &format!("checking trailing content after parsing '{data_type}'"),
-            ));
+            ))
         } else {
             Ok(data_type)
         }