Skip to content

Commit

Permalink
Add rule to push table function invocation into connector
Browse files Browse the repository at this point in the history
  • Loading branch information
kasiafi committed May 10, 2022
1 parent 7bb0aca commit 364af6b
Show file tree
Hide file tree
Showing 3 changed files with 90 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,7 @@
import io.trino.sql.planner.iterative.rule.ReplaceRedundantJoinWithSource;
import io.trino.sql.planner.iterative.rule.ReplaceWindowWithRowNumber;
import io.trino.sql.planner.iterative.rule.RewriteSpatialPartitioningAggregation;
import io.trino.sql.planner.iterative.rule.RewriteTableFunctionToTableScan;
import io.trino.sql.planner.iterative.rule.SimplifyCountOverConstant;
import io.trino.sql.planner.iterative.rule.SimplifyExpressions;
import io.trino.sql.planner.iterative.rule.SimplifyFilterPredicate;
Expand Down Expand Up @@ -605,6 +606,7 @@ public PlanOptimizers(
.add(new PushAggregationIntoTableScan(plannerContext, typeAnalyzer))
.add(new PushDistinctLimitIntoTableScan(plannerContext, typeAnalyzer))
.add(new PushTopNIntoTableScan(metadata))
.add(new RewriteTableFunctionToTableScan(plannerContext))
.build();
IterativeOptimizer pushIntoTableScanOptimizer = new IterativeOptimizer(
plannerContext,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,83 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package io.trino.sql.planner.iterative.rule;

import com.google.common.collect.ImmutableMap;
import io.trino.matching.Captures;
import io.trino.matching.Pattern;
import io.trino.metadata.TableHandle;
import io.trino.spi.connector.ColumnHandle;
import io.trino.spi.connector.TableFunctionApplicationResult;
import io.trino.spi.predicate.TupleDomain;
import io.trino.sql.PlannerContext;
import io.trino.sql.planner.Symbol;
import io.trino.sql.planner.iterative.Rule;
import io.trino.sql.planner.plan.TableFunctionNode;
import io.trino.sql.planner.plan.TableScanNode;

import java.util.List;
import java.util.Optional;

import static com.google.common.base.Preconditions.checkState;
import static io.trino.matching.Pattern.empty;
import static io.trino.sql.planner.plan.Patterns.sources;
import static io.trino.sql.planner.plan.Patterns.tableFunction;
import static java.util.Objects.requireNonNull;

public class RewriteTableFunctionToTableScan
implements Rule<TableFunctionNode>
{
private static final Pattern<TableFunctionNode> PATTERN = tableFunction()
.with(empty(sources()));

private final PlannerContext plannerContext;

public RewriteTableFunctionToTableScan(PlannerContext plannerContext)
{
this.plannerContext = requireNonNull(plannerContext, "plannerContext is null");
}

@Override
public Pattern<TableFunctionNode> getPattern()
{
return PATTERN;
}

@Override
public Result apply(TableFunctionNode tableFunctionNode, Captures captures, Context context)
{
Optional<TableFunctionApplicationResult<TableHandle>> result = plannerContext.getMetadata().applyTableFunction(context.getSession(), tableFunctionNode.getHandle());

if (result.isEmpty()) {
return Result.empty();
}

List<ColumnHandle> columnHandles = result.get().getColumnHandles();
checkState(tableFunctionNode.getOutputSymbols().size() == columnHandles.size(), "returned table does not match the node's output");
ImmutableMap.Builder<Symbol, ColumnHandle> assignments = ImmutableMap.builder();
for (int i = 0; i < columnHandles.size(); i++) {
assignments.put(tableFunctionNode.getOutputSymbols().get(i), columnHandles.get(i));
}

return Result.ofPlanNode(new TableScanNode(
tableFunctionNode.getId(),
result.get().getTableHandle(),
tableFunctionNode.getOutputSymbols(),
assignments.buildOrThrow(),
TupleDomain.all(),
Optional.empty(),
false,
Optional.empty()));
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -205,6 +205,11 @@ public static Pattern<PatternRecognitionNode> patternRecognition()
return typeOf(PatternRecognitionNode.class);
}

public static Pattern<TableFunctionNode> tableFunction()
{
return typeOf(TableFunctionNode.class);
}

public static Pattern<RowNumberNode> rowNumber()
{
return typeOf(RowNumberNode.class);
Expand Down

0 comments on commit 364af6b

Please sign in to comment.