Skip to content

Commit

Permalink
rebase
Browse files Browse the repository at this point in the history
  • Loading branch information
MohamedAbdeen21 committed May 4, 2024
1 parent eca91a5 commit 538b505
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion datafusion/optimizer/src/common_subexpr_eliminate.rs
Original file line number Diff line number Diff line change
Expand Up @@ -645,7 +645,7 @@ impl TreeNodeVisitor for ExprIdentifierVisitor<'_> {
let alias_symbol = format!("#{{{curr_expr_identifier}}}");

self.expr_set
.entry(curr_expr_identifier)
.entry(curr_expr_identifier.clone())
.or_insert_with(|| (expr.clone(), 0, data_type, alias_symbol))
.1 += 1;

Expand Down Expand Up @@ -1147,8 +1147,13 @@ mod test {
.build()?;

let expected = "Projection: test.a, test.b, test.c\
<<<<<<< HEAD
\n Filter: #{Int32(1) + test.a} - Int32(10) > #{Int32(1) + test.a}\
\n Projection: Int32(1) + test.a AS #{Int32(1) + test.a}, test.a, test.b, test.c\
=======
\n Filter: Int32(1) + test.a - Int32(10) > Int32(1) + test.a\
\n Projection: Int32(1) + test.a AS Int32(1) + test.a, test.a, test.b, test.c\
>>>>>>> 2a1f02109 (remove expr node accumulation)
\n TableScan: test";

assert_optimized_plan_eq(expected, &plan);
Expand Down

0 comments on commit 538b505

Please sign in to comment.