You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@vxquery.apache.org by pr...@apache.org on 2013/10/25 00:07:23 UTC
svn commit: r1535574 -
/incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java
Author: prestonc
Date: Thu Oct 24 22:07:22 2013
New Revision: 1535574
URL: http://svn.apache.org/r1535574
Log:
Update the interface based on reviewer's feedback in Hyracks.
Modified:
incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java
Modified: incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java
URL: http://svn.apache.org/viewvc/incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java?rev=1535574&r1=1535573&r2=1535574&view=diff
==============================================================================
--- incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java (original)
+++ incubator/vxquery/trunk/vxquery/vxquery-core/src/main/java/org/apache/vxquery/compiler/rewriter/rules/InlineReferenceVariablePolicy.java Thu Oct 24 22:07:22 2013
@@ -16,7 +16,6 @@ package org.apache.vxquery.compiler.rewr
import java.util.ArrayList;
import java.util.List;
-import java.util.Set;
import org.apache.commons.lang3.mutable.Mutable;
@@ -25,7 +24,6 @@ import edu.uci.ics.hyracks.algebricks.co
import edu.uci.ics.hyracks.algebricks.core.algebra.base.ILogicalPlan;
import edu.uci.ics.hyracks.algebricks.core.algebra.base.LogicalExpressionTag;
import edu.uci.ics.hyracks.algebricks.core.algebra.base.LogicalOperatorTag;
-import edu.uci.ics.hyracks.algebricks.core.algebra.functions.FunctionIdentifier;
import edu.uci.ics.hyracks.algebricks.core.algebra.operators.logical.AbstractLogicalOperator;
import edu.uci.ics.hyracks.algebricks.core.algebra.operators.logical.SubplanOperator;
import edu.uci.ics.hyracks.algebricks.rewriter.rules.InlineVariablesRule;
@@ -57,7 +55,7 @@ import edu.uci.ics.hyracks.algebricks.re
public class InlineReferenceVariablePolicy implements InlineVariablesRule.IInlineVariablePolicy {
@Override
- public boolean addExpressionToInlineMap(ILogicalExpression expr, Set<FunctionIdentifier> doNotInlineFuncs) {
+ public boolean isCandidateForInlining(ILogicalExpression expr) {
if (expr.getExpressionTag() == LogicalExpressionTag.VARIABLE) {
return true;
}
@@ -84,7 +82,7 @@ public class InlineReferenceVariablePoli
}
@Override
- public boolean transformOperator(AbstractLogicalOperator op) {
+ public boolean isCanidateInlineTarget(AbstractLogicalOperator op) {
return true;
}