[
https://issues.apache.org/jira/browse/PIG-3567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13822131#comment-13822131
]
Aniket Mokashi commented on PIG-3567:
-------------------------------------
Fixed it by making LogicalPlanPrinter write to stream recursively (patch
attached). There are no existing tests for verifying printing for logical plan.
Do we need to add tests for this(changes are non-trivial)? I verified with a
few scripts that logical plan remains the same.
> LogicalPlanPrinter throws OOM for large scripts
> -----------------------------------------------
>
> Key: PIG-3567
> URL: https://issues.apache.org/jira/browse/PIG-3567
> Project: Pig
> Issue Type: Bug
> Reporter: Aniket Mokashi
> Assignee: Aniket Mokashi
> Attachments: PIG-3567.patch
>
>
> As mentioned in PIG-3455, LogicalPlanPrinter throws OOM for large scripts.
> Problem is LogicalPlanPrinter's visit method generates a large string before
> its written to the PrintStream.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
