Skip to content

Commit

Permalink
wip: remove dead code
Browse files Browse the repository at this point in the history
  • Loading branch information
matthewgapp committed Nov 19, 2023
1 parent 7550b29 commit ce3cdcd
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions datafusion/core/src/physical_optimizer/enforce_distribution.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1114,30 +1114,6 @@ fn remove_dist_changing_operators(
})
}

// /// Changes each child of the `dist_context.plan` such that they no longer
// /// use order preserving variants, if no ordering is required at the output
// /// of the physical plan (there is no global ordering requirement by the query).
// fn update_plan_to_remove_unnecessary_final_order(
// dist_context: DistributionContext,
// ) -> Result<Arc<dyn ExecutionPlan>> {
// let DistributionContext {
// plan,
// distribution_onwards,
// ..
// } = dist_context;
// let new_children = izip!(plan.children(), distribution_onwards)
// .map(|(mut child, mut dist_onward)| {
// replace_order_preserving_variants(&mut child, &mut dist_onward)?;
// Ok(child)
// })
// .collect::<Result<Vec<_>>>()?;
// if !new_children.is_empty() {
// plan.with_new_children(new_children)
// } else {
// Ok(plan)
// }
// }

/// Updates the physical plan `input` by using `dist_onward` replace order preserving operator variants
/// with their corresponding operators that do not preserve order. It is a wrapper for `replace_order_preserving_variants_helper`
fn replace_order_preserving_variants(
Expand Down

0 comments on commit ce3cdcd

Please sign in to comment.