From d67e319b9c16fd60aff7574b25352b7c48d142d5 Mon Sep 17 00:00:00 2001 From: Orson Peters Date: Thu, 19 Dec 2024 17:53:13 +0100 Subject: [PATCH] refactor(rust): Temporarily disable common subplan elim for new-streaming --- crates/polars-lazy/src/frame/mod.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/crates/polars-lazy/src/frame/mod.rs b/crates/polars-lazy/src/frame/mod.rs index 154726e6a27e..d0860f3faccd 100644 --- a/crates/polars-lazy/src/frame/mod.rs +++ b/crates/polars-lazy/src/frame/mod.rs @@ -601,11 +601,15 @@ impl LazyFrame { opt_state &= !OptFlags::COMM_SUBPLAN_ELIM; } - // The new streaming engine can't deal with the way the common - // subexpression elimination adds length-incorrect with_columns. #[cfg(feature = "cse")] if new_streaming { + // The new streaming engine can't deal with the way the common + // subexpression elimination adds length-incorrect with_columns. opt_state &= !OptFlags::COMM_SUBEXPR_ELIM; + + // The new streaming engine can't yet deal with the cache nodes + // introduced by common subplan elimination. + opt_state &= !OptFlags::COMM_SUBPLAN_ELIM; } let lp_top = optimize(