Skip to content

Commit

Permalink
Remove redundant constant removal loop
Browse files Browse the repository at this point in the history
  • Loading branch information
gokselk committed Dec 17, 2024
1 parent 2721609 commit 291257f
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions datafusion/physical-expr/src/equivalence/properties.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1858,12 +1858,6 @@ fn calculate_union_binary(
})
.collect::<Vec<_>>();

// Remove any constants that are shared in both outputs (avoid double counting them)
for c in &constants {
lhs = lhs.remove_constant(c);
rhs = rhs.remove_constant(c);
}

// Next, calculate valid orderings for the union by searching for prefixes
// in both sides.
let mut orderings = UnionEquivalentOrderingBuilder::new();
Expand Down

0 comments on commit 291257f

Please sign in to comment.