Rename listTableLayouts method

More accurately, constructs a set of alternate plans with the filter pushed into the table scan based on available table layouts.
trinodb · Mar 7, 2019 · d7cde8c · d7cde8c
1 parent cbf8c50
commit d7cde8c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/presto-main/src/main/java/io/prestosql/sql/planner/iterative/rule/PickTableLayout.java b/presto-main/src/main/java/io/prestosql/sql/planner/iterative/rule/PickTableLayout.java
@@ -228,7 +228,7 @@ private static PlanNode planTableScan(
             SqlParser parser,
             DomainTranslator domainTranslator)
     {
-        return listTableLayouts(
+        return pushFilterIntoTableScan(
                 node,
                 predicate,
                 false,
@@ -241,7 +241,7 @@ private static PlanNode planTableScan(
                 .get(0);
     }
 
-    public static List<PlanNode> listTableLayouts(
+    public static List<PlanNode> pushFilterIntoTableScan(
             TableScanNode node,
             Expression predicate,
             boolean pruneWithPredicateExpression,

diff --git a/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/AddExchanges.java b/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/AddExchanges.java
@@ -540,7 +540,7 @@ else if (redistributeWrites) {
 
         private PlanWithProperties planTableScan(TableScanNode node, Expression predicate, PreferredProperties preferredProperties)
         {
-            List<PlanNode> possiblePlans = PickTableLayout.listTableLayouts(node, predicate, true, session, types, idAllocator, metadata, parser, domainTranslator);
+            List<PlanNode> possiblePlans = PickTableLayout.pushFilterIntoTableScan(node, predicate, true, session, types, idAllocator, metadata, parser, domainTranslator);
             List<PlanWithProperties> possiblePlansWithProperties = possiblePlans.stream()
                     .map(planNode -> new PlanWithProperties(planNode, derivePropertiesRecursively(planNode)))
                     .collect(toImmutableList());