Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

Commit ef1815c

Browse files
committed
fix(df-repr): enable subquery rules (#223)
Signed-off-by: Alex Chi <[email protected]>
1 parent 3a44cea commit ef1815c

File tree

3 files changed

+156
-239
lines changed

3 files changed

+156
-239
lines changed

optd-datafusion-repr/src/lib.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -83,11 +83,11 @@ impl DatafusionOptimizer {
8383
Arc::new(rules::EliminateLimitRule::new()),
8484
Arc::new(rules::EliminateDuplicatedSortExprRule::new()),
8585
Arc::new(rules::EliminateDuplicatedAggExprRule::new()),
86-
// Arc::new(rules::DepJoinEliminate::new()),
87-
// Arc::new(rules::DepInitialDistinct::new()),
88-
// Arc::new(rules::DepJoinPastProj::new()),
89-
// Arc::new(rules::DepJoinPastFilter::new()),
90-
// Arc::new(rules::DepJoinPastAgg::new()),
86+
Arc::new(rules::DepJoinEliminate::new()),
87+
Arc::new(rules::DepInitialDistinct::new()),
88+
Arc::new(rules::DepJoinPastProj::new()),
89+
Arc::new(rules::DepJoinPastFilter::new()),
90+
Arc::new(rules::DepJoinPastAgg::new()),
9191
Arc::new(rules::ProjectMergeRule::new()),
9292
Arc::new(rules::FilterMergeRule::new()),
9393
]

optd-datafusion-repr/src/rules.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ mod joins;
66
mod macros;
77
mod physical;
88
mod project_transpose;
9-
// mod subquery;
9+
mod subquery;
1010

1111
pub use eliminate_duplicated_expr::*;
1212
pub use eliminate_limit::*;
@@ -15,6 +15,6 @@ pub use filter_pushdown::*;
1515
pub use joins::*;
1616
pub use physical::PhysicalConversionRule;
1717
pub use project_transpose::*;
18-
// pub use subquery::{
19-
// DepInitialDistinct, DepJoinEliminate, DepJoinPastAgg, DepJoinPastFilter, DepJoinPastProj,
20-
// };
18+
pub use subquery::{
19+
DepInitialDistinct, DepJoinEliminate, DepJoinPastAgg, DepJoinPastFilter, DepJoinPastProj,
20+
};

0 commit comments

Comments
 (0)