Skip to content

Commit 047f937

Browse files
committed
fix clippy
1 parent b3f36f0 commit 047f937

File tree

3 files changed

+3
-9
lines changed

3 files changed

+3
-9
lines changed

datafusion/physical-optimizer/src/sanity_checker.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ pub fn check_filtered_batch_handle(
114114
for child in plan.children() {
115115
if !child.output_filtered_batches() {
116116
let plan_str = get_plan_string(&plan);
117-
let child_str = get_plan_string(&child);
117+
let child_str = get_plan_string(child);
118118
return plan_err!(
119119
"Plan {:?} required filtered batches but its child {:?} won't produce them",
120120
plan_str,

datafusion/physical-plan/src/aggregates/mod.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1020,10 +1020,7 @@ impl ExecutionPlan for AggregateExec {
10201020
}
10211021

10221022
fn requried_filtered_input(&self) -> bool {
1023-
match self.mode() {
1024-
AggregateMode::FinalPartitioned(HashPartitionMode::SelectionBitmap) => true,
1025-
_ => false,
1026-
}
1023+
matches!(self.mode(), AggregateMode::FinalPartitioned(HashPartitionMode::SelectionBitmap))
10271024
}
10281025
}
10291026

datafusion/physical-plan/src/repartition/mod.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -653,10 +653,7 @@ impl ExecutionPlan for RepartitionExec {
653653
}
654654

655655
fn output_filtered_batches(&self) -> bool {
656-
match self.partitioning() {
657-
Partitioning::HashSelectionBitmap(_, _) => true,
658-
_ => false,
659-
}
656+
matches!(self.partitioning(),Partitioning::HashSelectionBitmap(_, _))
660657
}
661658

662659
fn execute(

0 commit comments

Comments
 (0)