Duplicate code in LOPrinter.java
--------------------------------

                 Key: PIG-1422
                 URL: https://issues.apache.org/jira/browse/PIG-1422
             Project: Pig
          Issue Type: Improvement
          Components: impl
    Affects Versions: 0.8.0
            Reporter: Jeff Zhang
            Assignee: Jeff Zhang
            Priority: Minor
             Fix For: 0.8.0


Today when I go through the source code , I find the following duplicate code 
in LOPrinter.java  (line 153 --- line 170)
{code}
           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 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));
                    }
                }
            }
{code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to