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 3, 2024 · d9b07d9 · d9b07d9
1 parent f40062b
commit d9b07d9
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,
         }
     }