You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by da...@apache.org on 2010/11/19 23:23:52 UTC
svn commit: r1037057 - in /pig/branches/branch-0.8: CHANGES.txt
src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java
Author: daijy
Date: Fri Nov 19 22:23:51 2010
New Revision: 1037057
URL: http://svn.apache.org/viewvc?rev=1037057&view=rev
Log:
PIG-1716: New logical plan: LogToPhyTranslationVisitor should translate the structure for regex optimization
Modified:
pig/branches/branch-0.8/CHANGES.txt
pig/branches/branch-0.8/src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java
Modified: pig/branches/branch-0.8/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.8/CHANGES.txt?rev=1037057&r1=1037056&r2=1037057&view=diff
==============================================================================
--- pig/branches/branch-0.8/CHANGES.txt (original)
+++ pig/branches/branch-0.8/CHANGES.txt Fri Nov 19 22:23:51 2010
@@ -207,6 +207,8 @@ PIG-1309: Map-side Cogroup (ashutoshc)
BUG FIXES
+PIG-1716: New logical plan: LogToPhyTranslationVisitor should translate the structure for regex optimization (daijy)
+
PIG-1740: Fix SVN location in setup doc (chandec via olgan)
PIG-1719: New logical plan: FieldSchema generation for BinCond is wrong (daijy)
Modified: pig/branches/branch-0.8/src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.8/src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java?rev=1037057&r1=1037056&r2=1037057&view=diff
==============================================================================
--- pig/branches/branch-0.8/src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java (original)
+++ pig/branches/branch-0.8/src/org/apache/pig/newplan/logical/expression/ExpToPhyTranslationVisitor.java Fri Nov 19 22:23:51 2010
@@ -415,6 +415,11 @@ public class ExpToPhyTranslationVisitor
BinaryExpressionOperator exprOp = new PORegexp(new OperatorKey(DEFAULT_SCOPE, nodeGen.getNextNodeId(DEFAULT_SCOPE)));
attachBinaryExpressionOperator(op, exprOp);
+
+ List<Operator> successors = op.getPlan().getSuccessors(op);
+ if (successors.get(1) instanceof org.apache.pig.newplan.logical.expression.ConstantExpression) {
+ ((PORegexp)exprOp).setConstExpr(true);
+ }
}
@Override