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())); 
             }


Reply via email to