Skip to content

Commit 1b36400

Browse files
authored
Move Pruning Logic to a Dedicated datafusion-pruning Crate for Improved Modularity (#16549)
* feat: add datafusion-pruning crate * fix pruning tests * refactor: remove unused pruning module
1 parent 4a4ffd7 commit 1b36400

File tree

6 files changed

+49
-1
lines changed

6 files changed

+49
-1
lines changed

Cargo.lock

Lines changed: 18 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ members = [
4242
"datafusion/physical-expr",
4343
"datafusion/physical-expr-common",
4444
"datafusion/physical-optimizer",
45+
"datafusion/pruning",
4546
"datafusion/physical-plan",
4647
"datafusion/proto",
4748
"datafusion/proto/gen",
@@ -139,6 +140,7 @@ datafusion-physical-optimizer = { path = "datafusion/physical-optimizer", versio
139140
datafusion-physical-plan = { path = "datafusion/physical-plan", version = "48.0.0" }
140141
datafusion-proto = { path = "datafusion/proto", version = "48.0.0" }
141142
datafusion-proto-common = { path = "datafusion/proto-common", version = "48.0.0" }
143+
datafusion-pruning = { path = "datafusion/pruning", version = "48.0.0" }
142144
datafusion-session = { path = "datafusion/session", version = "48.0.0" }
143145
datafusion-spark = { path = "datafusion/spark", version = "48.0.0" }
144146
datafusion-sql = { path = "datafusion/sql", version = "48.0.0" }

datafusion/physical-optimizer/Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ datafusion-expr-common = { workspace = true, default-features = true }
4646
datafusion-physical-expr = { workspace = true }
4747
datafusion-physical-expr-common = { workspace = true }
4848
datafusion-physical-plan = { workspace = true }
49+
datafusion-pruning = { workspace = true }
4950
itertools = { workspace = true }
5051
log = { workspace = true }
5152
recursive = { workspace = true, optional = true }

datafusion/physical-optimizer/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ pub mod limited_distinct_aggregation;
3838
pub mod optimizer;
3939
pub mod output_requirements;
4040
pub mod projection_pushdown;
41-
pub mod pruning;
41+
pub use datafusion_pruning as pruning;
4242
pub mod sanity_checker;
4343
pub mod topk_aggregation;
4444
pub mod update_aggr_exprs;

datafusion/pruning/Cargo.toml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
[package]
2+
name = "datafusion-pruning"
3+
description = "DataFusion Pruning Logic"
4+
version = { workspace = true }
5+
edition = { workspace = true }
6+
homepage = { workspace = true }
7+
repository = { workspace = true }
8+
license = { workspace = true }
9+
authors = { workspace = true }
10+
11+
[lints]
12+
workspace = true
13+
14+
[dependencies]
15+
arrow = { workspace = true }
16+
arrow-schema = { workspace = true }
17+
datafusion-common = { workspace = true, default-features = true }
18+
datafusion-expr-common = { workspace = true, default-features = true }
19+
datafusion-physical-expr = { workspace = true }
20+
datafusion-physical-expr-common = { workspace = true }
21+
datafusion-physical-plan = { workspace = true }
22+
log = { workspace = true }
23+
24+
[dev-dependencies]
25+
datafusion-expr = { workspace = true }
26+
datafusion-functions-nested = { workspace = true }
27+
insta = { workspace = true }
File renamed without changes.

0 commit comments

Comments
 (0)