From d80735e256b52e1ec19c95bd3560969a72cf53bc Mon Sep 17 00:00:00 2001 From: Victor Barua Date: Sat, 20 Dec 2025 13:15:21 -0800 Subject: [PATCH] refactor: delete unused code --- .../substrait/isthmus/SubstraitToCalcite.java | 31 ------------------- 1 file changed, 31 deletions(-) diff --git a/isthmus/src/main/java/io/substrait/isthmus/SubstraitToCalcite.java b/isthmus/src/main/java/io/substrait/isthmus/SubstraitToCalcite.java index 772a3e192..a0c5132e4 100644 --- a/isthmus/src/main/java/io/substrait/isthmus/SubstraitToCalcite.java +++ b/isthmus/src/main/java/io/substrait/isthmus/SubstraitToCalcite.java @@ -3,16 +3,10 @@ import io.substrait.extension.SimpleExtension; import io.substrait.isthmus.SubstraitRelNodeConverter.Context; import io.substrait.plan.Plan; -import io.substrait.relation.NamedScan; import io.substrait.relation.Rel; -import io.substrait.relation.RelCopyOnWriteVisitor; -import io.substrait.type.NamedStruct; import io.substrait.util.EmptyVisitationContext; import java.util.ArrayList; -import java.util.HashMap; import java.util.List; -import java.util.Map; -import java.util.Optional; import org.apache.calcite.jdbc.CalciteSchema; import org.apache.calcite.prepare.Prepare; import org.apache.calcite.rel.RelNode; @@ -229,29 +223,4 @@ private Pair renameFields( return Pair.of(currentIndex, type); } } - - private static class NamedStructGatherer extends RelCopyOnWriteVisitor { - Map, NamedStruct> tableMap; - - private NamedStructGatherer() { - super(); - this.tableMap = new HashMap<>(); - } - - public static Map, NamedStruct> gatherTables(Rel rel) { - NamedStructGatherer visitor = new NamedStructGatherer(); - rel.accept(visitor, EmptyVisitationContext.INSTANCE); - return visitor.tableMap; - } - - @Override - public Optional visit(NamedScan namedScan, EmptyVisitationContext context) { - Optional result = super.visit(namedScan, context); - - List tableName = namedScan.getNames(); - tableMap.put(tableName, namedScan.getInitialSchema()); - - return result; - } - } }