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/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)
  }