refactor(cubesql): Remove Extension from LogicalPlanLanguage::CubeScan

Extensions is used meaningfully only for CubeScan, but not for CubeScanWrapper. Both are first-class logical plan nodes for us, so Extension is not helpful in egraph
cube-js · Oct 4, 2024 · 499510b · 499510b
1 parent da70a8e
commit 499510b
Show file tree

Hide file tree

Showing 3 changed files with 468 additions and 530 deletions.
diff --git a/rust/cubesql/cubesql/src/compile/rewrite/analysis.rs b/rust/cubesql/cubesql/src/compile/rewrite/analysis.rs
@@ -1221,7 +1221,6 @@ impl LogicalPlanAnalysis {
                 Some(c.to_string())
             }
             LogicalPlanLanguage::CubeScan(params) => cube_reference(params[0]),
-            LogicalPlanLanguage::Extension(params) => cube_reference(params[0]),
             _ => None,
         }
     }