Consolidate push_down_projection tests under optimize_projections #9978
Assignees
Labels
Comments
Jefffrey
added
enhancement
|
take |
kavirajk
added a commit
to kavirajk/datafusion
that referenced
this issue
Apr 13, 2024
…ze_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 <kavirajkanagaraj@gmail.com>
kavirajk
added a commit
to kavirajk/datafusion
that referenced
this issue
Apr 14, 2024
…ze_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 <kavirajkanagaraj@gmail.com>
alamb
pushed a commit
that referenced
this issue
Apr 15, 2024
…ze_projections.rs` (#10071) * cleanup(tests): Move tests from `push_down_projections.rs` to `optimize_projections.rs` Fixes: #9978 The PR #8340 removed `push_down_projections.rs` in favour of `optimize_projections.rs`. This PR moves the tests as well. Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com> * remove the file `push_down_projection.rs` Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com> * Fix method signatures that are broken by other PRs Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com> * remove `push_down_projections.rs` from `lib.rs` Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com> --------- Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem or challenge?
#8340 removed push_down_projection rules in favour of optimize_projections
Describe the solution you'd like
Move tests in push_down_projection.rs to optimize_projections.rs and remove the
push_down_projection.rsfileDescribe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: