More clippy

viirya · Dec 29, 2023 · edc825e · edc825e
1 parent c18464a
commit edc825e
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/datafusion/core/src/physical_optimizer/enforce_distribution.rs b/datafusion/core/src/physical_optimizer/enforce_distribution.rs
@@ -1410,7 +1410,7 @@ impl DistributionContext {
 
 impl TreeNode for DistributionContext {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children_nodes.iter().map(|c| c.clone()).collect()
+        self.children_nodes.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>
@@ -1474,7 +1474,7 @@ impl PlanWithKeyRequirements {
 
 impl TreeNode for PlanWithKeyRequirements {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children.iter().map(|c| c.clone()).collect()
+        self.children.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>

diff --git a/datafusion/core/src/physical_optimizer/enforce_sorting.rs b/datafusion/core/src/physical_optimizer/enforce_sorting.rs
@@ -146,7 +146,7 @@ impl PlanWithCorrespondingSort {
 
 impl TreeNode for PlanWithCorrespondingSort {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children_nodes.iter().map(|c| c.clone()).collect()
+        self.children_nodes.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>
@@ -227,7 +227,7 @@ impl PlanWithCorrespondingCoalescePartitions {
 
 impl TreeNode for PlanWithCorrespondingCoalescePartitions {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children_nodes.iter().map(|c| c.clone()).collect()
+        self.children_nodes.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>

diff --git a/datafusion/core/src/physical_optimizer/pipeline_checker.rs b/datafusion/core/src/physical_optimizer/pipeline_checker.rs
@@ -92,7 +92,7 @@ impl PipelineStatePropagator {
 
 impl TreeNode for PipelineStatePropagator {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children.iter().map(|c| c.clone()).collect()
+        self.children.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>

diff --git a/datafusion/core/src/physical_optimizer/replace_with_order_preserving_variants.rs b/datafusion/core/src/physical_optimizer/replace_with_order_preserving_variants.rs
@@ -105,7 +105,7 @@ impl OrderPreservationContext {
 
 impl TreeNode for OrderPreservationContext {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children_nodes.iter().map(|c| c.clone()).collect()
+        self.children_nodes.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>

diff --git a/datafusion/core/src/physical_optimizer/sort_pushdown.rs b/datafusion/core/src/physical_optimizer/sort_pushdown.rs
@@ -72,7 +72,7 @@ impl SortPushDown {
 
 impl TreeNode for SortPushDown {
     fn children_nodes(&self) -> Vec<Self> {
-        self.children_nodes.iter().map(|c| c.clone()).collect()
+        self.children_nodes.iter().cloned().collect()
     }
 
     fn map_children<F>(mut self, transform: F) -> Result<Self>