Skip to content

Commit

Permalink
cleanup(tests): Move tests from push_down_projections.rs to `optimi…
Browse files Browse the repository at this point in the history
…ze_projections.rs` (apache#10071)

* cleanup(tests): Move tests from `push_down_projections.rs` to `optimize_projections.rs`

Fixes: apache#9978

The PR apache#8340 removed `push_down_projections.rs` in favour of `optimize_projections.rs`. This PR moves the tests as well.

Signed-off-by: Kaviraj <[email protected]>

* remove the file `push_down_projection.rs`

Signed-off-by: Kaviraj <[email protected]>

* Fix method signatures that are broken by other PRs

Signed-off-by: Kaviraj <[email protected]>

* remove `push_down_projections.rs` from `lib.rs`

Signed-off-by: Kaviraj <[email protected]>

---------

Signed-off-by: Kaviraj <[email protected]>
  • Loading branch information
kavirajk authored Apr 15, 2024
1 parent 34eda15 commit 060e67e
Show file tree
Hide file tree
Showing 3 changed files with 631 additions and 667 deletions.
1 change: 0 additions & 1 deletion datafusion/optimizer/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ pub mod optimizer;
pub mod propagate_empty_relation;
pub mod push_down_filter;
pub mod push_down_limit;
pub mod push_down_projection;
pub mod replace_distinct_aggregate;
pub mod rewrite_disjunctive_predicate;
pub mod scalar_subquery_to_join;
Expand Down
Loading

0 comments on commit 060e67e

Please sign in to comment.