[jira] Commented: (PIG-634) When POUnion is one of the roots of a map plan, POUnion.getNext() gives a null pointer exception
[
https://issues.apache.org/jira/browse/PIG-634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12667439#action_12667439
]
Pradeep Kamath commented on PIG-634:
Patch committed
> When POUnion is one of the roots of a map plan, POUnion.getNext() gives a
> null pointer exception
>
>
> Key: PIG-634
> URL: https://issues.apache.org/jira/browse/PIG-634
> Project: Pig
> Issue Type: Bug
>Affects Versions: types_branch
>Reporter: Pradeep Kamath
>Assignee: Pradeep Kamath
> Fix For: types_branch
>
> Attachments: PIG-634.patch
>
>
> POUnion.getnext() gives a null pointer exception in the following scenario
> (pasted from a code comment explaining the fix for this issue). If a script
> results in a plan like the one below, currently POUnion.getNext() gives a
> null pointer exception
> {noformat}
>
> // POUnion
> // |
> // |--POLocalRearrange
> // ||
> // ||-POUnion (root 2)--> This union's getNext() can lead
> the code here
> // |
> // |--POLocalRearrange (root 1)
>
> // The inner POUnion above is a root in the plan which has 2
> roots.
> // So these 2 roots would have input coming from different
> input
> // sources (dfs files). So certain maps would be working on
> input only
> // meant for "root 1" above and some maps would work on input
> // meant only for "root 2". In the former case, "root 2" would
> // neither get input attached to it nor does it have
> predecessors
> {noformat}
> A script which can cause a plan like above is:
> {code}
> a = load 'xyz';
> b = load 'abc';
> c = union a,b;
> d = load 'def';
> e = cogroup c by $0 inner , d by $0 inner;
> dump e;
> {code}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (PIG-634) When POUnion is one of the roots of a map plan, POUnion.getNext() gives a null pointer exception
[
https://issues.apache.org/jira/browse/PIG-634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12667335#action_12667335
]
Olga Natkovich commented on PIG-634:
+1 on the patch. Great catch!
> When POUnion is one of the roots of a map plan, POUnion.getNext() gives a
> null pointer exception
>
>
> Key: PIG-634
> URL: https://issues.apache.org/jira/browse/PIG-634
> Project: Pig
> Issue Type: Bug
>Affects Versions: types_branch
>Reporter: Pradeep Kamath
>Assignee: Pradeep Kamath
> Fix For: types_branch
>
> Attachments: PIG-634.patch
>
>
> POUnion.getnext() gives a null pointer exception in the following scenario
> (pasted from a code comment explaining the fix for this issue). If a script
> results in a plan like the one below, currently POUnion.getNext() gives a
> null pointer exception
> {noformat}
>
> // POUnion
> // |
> // |--POLocalRearrange
> // ||
> // ||-POUnion (root 2)--> This union's getNext() can lead
> the code here
> // |
> // |--POLocalRearrange (root 1)
>
> // The inner POUnion above is a root in the plan which has 2
> roots.
> // So these 2 roots would have input coming from different
> input
> // sources (dfs files). So certain maps would be working on
> input only
> // meant for "root 1" above and some maps would work on input
> // meant only for "root 2". In the former case, "root 2" would
> // neither get input attached to it nor does it have
> predecessors
> {noformat}
> A script which can cause a plan like above is:
> {code}
> a = load 'xyz';
> b = load 'abc';
> c = union a,b;
> d = load 'def';
> e = cogroup c by $0 inner , d by $0 inner;
> dump e;
> {code}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
