[
https://issues.apache.org/jira/browse/PIG-3508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13789673#comment-13789673
]
Daniel Dai commented on PIG-3508:
---------------------------------
We shall change it anyway even rollback PIG-3419. Depends on compilePp() to
optimize the logical plan is very weird.
> 'explain' now showing logical plan BEFORE the necessary optimization
> (ImplicitSplitInserter, DuplicateForEachColumnRewrite,etc)
> -------------------------------------------------------------------------------------------------------------------------------
>
> Key: PIG-3508
> URL: https://issues.apache.org/jira/browse/PIG-3508
> Project: Pig
> Issue Type: Bug
> Components: parser
> Affects Versions: tez-branch, 0.13.0
> Reporter: Koji Noguchi
> Priority: Minor
>
> Probably PIG-3419 changed the order of the call such that now 'explain' is
> showing the logicalPlan before the LogicalPlanOptimizer.optimize() is being
> called.
> Before we had
> {noformat}
> 1030 PhysicalPlan pp = compilePp();
> 1031 currDAG.lp.explain(lps, format, verbose);
> {noformat}
> where LogicalPlanOptimizer.optimize() was called from compilePp().
--
This message was sent by Atlassian JIRA
(v6.1#6144)
