[
https://issues.apache.org/jira/browse/PIG-4455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14355180#comment-14355180
]
Rohini Palaniswamy commented on PIG-4455:
-----------------------------------------
bq. TestMRCompiler passed, and notice that TestMRCompiler use PlanPrinter
instead of MRPrinter
Saw that. But MROperPlan.toString() used MRPrinter. So was not sure if it was
being called.
+1
> Should use DependencyOrderWalker instead of DepthFirstWalker in MRPrinter
> -------------------------------------------------------------------------
>
> Key: PIG-4455
> URL: https://issues.apache.org/jira/browse/PIG-4455
> Project: Pig
> Issue Type: Improvement
> Reporter: Jeff Zhang
> Attachments: PIG-4455-1.patch, PIG-4455-2.patch
>
>
> Using DependencyOrderWalker is much better than DepthFirstWalker, and it
> would be better to also print the mr job dependency like TezGraphPrinter
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
