You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by zj...@apache.org on 2010/05/21 03:47:23 UTC
svn commit: r946852 - in /hadoop/pig/trunk: CHANGES.txt
src/org/apache/pig/impl/logicalLayer/LOPrinter.java
Author: zjffdu
Date: Fri May 21 01:47:23 2010
New Revision: 946852
URL: http://svn.apache.org/viewvc?rev=946852&view=rev
Log:
PIG-1422: Duplicate code in LOPrinter.java
Modified:
hadoop/pig/trunk/CHANGES.txt
hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/LOPrinter.java
Modified: hadoop/pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/CHANGES.txt?rev=946852&r1=946851&r2=946852&view=diff
==============================================================================
--- hadoop/pig/trunk/CHANGES.txt (original)
+++ hadoop/pig/trunk/CHANGES.txt Fri May 21 01:47:23 2010
@@ -24,6 +24,8 @@ INCOMPATIBLE CHANGES
IMPROVEMENTS
+PIG-1422: Duplicate code in LOPrinter.java (zjffdu)
+
PIG-1420: Make CONCAT act on all fields of a tuple, instead of just the first two fields of a tuple (rjurney via dvryaboy)
PIG-1408: Annotate explain plans with aliases (rding)
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/LOPrinter.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/LOPrinter.java?rev=946852&r1=946851&r2=946852&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/LOPrinter.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/LOPrinter.java Fri May 21 01:47:23 2010
@@ -159,15 +159,6 @@ public class LOPrinter extends LOVisitor
}
}
}
- else if(node instanceof LOJoin){
- MultiMap<LogicalOperator, LogicalPlan> plans = ((LOJoin)node).getJoinPlans();
- for (LogicalOperator lo : plans.keySet()) {
- // Visit the associated plans
- for (LogicalPlan plan : plans.get(lo)) {
- sb.append(planString(plan));
- }
- }
- }
else if(node instanceof LOSort){
sb.append(planString(((LOSort)node).getSortColPlans()));
}