Taewoo Kim has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1273
Change subject: Fixed UsedVariableVisitor for the Splitoperator
......................................................................
Fixed UsedVariableVisitor for the Splitoperator
- The used variables of SplitOperator and ReplicateOperator
now don't include the used variables from their output(s).
Change-Id: Iae861c1339cf00c5a53aa2b5b5b14113ec53484b
---
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/UsedVariableVisitor.java
1 file changed, 1 insertion(+), 6 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/73/1273/1
diff --git
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/UsedVariableVisitor.java
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/UsedVariableVisitor.java
index cfe2a37..3daa00f 100644
---
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/UsedVariableVisitor.java
+++
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/UsedVariableVisitor.java
@@ -428,17 +428,12 @@
@Override
public Void visitReplicateOperator(ReplicateOperator op, Void arg) throws
AlgebricksException {
- for (Mutable<ILogicalOperator> outputOp : op.getOutputs()) {
- VariableUtilities.getUsedVariables(outputOp.getValue(),
usedVariables);
- }
return null;
}
@Override
public Void visitSplitOperator(SplitOperator op, Void arg) throws
AlgebricksException {
- for (Mutable<ILogicalOperator> outputOp : op.getOutputs()) {
- VariableUtilities.getUsedVariables(outputOp.getValue(),
usedVariables);
- }
+ op.getBranchingExpression().getValue().getUsedVariables(usedVariables);
return null;
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/1273
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iae861c1339cf00c5a53aa2b5b5b14113ec53484b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim <[email protected]>
