diff --git a/src/daft-logical-plan/src/optimization/optimizer.rs b/src/daft-logical-plan/src/optimization/optimizer.rs index 68521b8a0d..bb23d5d995 100644 --- a/src/daft-logical-plan/src/optimization/optimizer.rs +++ b/src/daft-logical-plan/src/optimization/optimizer.rs @@ -7,7 +7,7 @@ use super::{ logical_plan_tracker::LogicalPlanTracker, rules::{ DropRepartition, EliminateCrossJoin, LiftProjectFromAgg, OptimizerRule, PushDownFilter, - PushDownLimit, PushDownProjection, SimplifyExpressionsRule, SplitActorPoolProjects, + PushDownLimit, PushDownProjection, SplitActorPoolProjects, }, }; use crate::LogicalPlan; @@ -94,10 +94,11 @@ impl Optimizer { let mut rule_batches = Vec::new(); // we want to simplify expressions first to make the rest of the rules easier - rule_batches.push(RuleBatch::new( - vec![Box::new(SimplifyExpressionsRule::new())], - RuleExecutionStrategy::FixedPoint(Some(3)), - )); + // dbg + // rule_batches.push(RuleBatch::new( + // vec![Box::new(SimplifyExpressionsRule::new())], + // RuleExecutionStrategy::FixedPoint(Some(3)), + // )); // --- Split ActorPoolProjection nodes from Project nodes --- // This is feature-flagged behind DAFT_ENABLE_ACTOR_POOL_PROJECTIONS=1 diff --git a/src/daft-logical-plan/src/optimization/rules/mod.rs b/src/daft-logical-plan/src/optimization/rules/mod.rs index 5829331e44..335c4320be 100644 --- a/src/daft-logical-plan/src/optimization/rules/mod.rs +++ b/src/daft-logical-plan/src/optimization/rules/mod.rs @@ -15,5 +15,6 @@ pub use push_down_filter::PushDownFilter; pub use push_down_limit::PushDownLimit; pub use push_down_projection::PushDownProjection; pub use rule::OptimizerRule; +#[allow(unused_imports)] pub use simplify_expressions::SimplifyExpressionsRule; pub use split_actor_pool_projects::SplitActorPoolProjects;