try disabling rule

Eventual-Inc · Nov 21, 2024 · 1edcb7b · 1edcb7b
1 parent 5d7853e
commit 1edcb7b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git 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
@@ -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;