[ 
https://issues.apache.org/jira/browse/PIG-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12591923#action_12591923
 ] 

Santhosh Srinivasan commented on PIG-158:
-----------------------------------------

Pi,

In principle, I agree with your views on nesting. I see your concern on having 
nested operators with some of them floating around. With the nested plans, we 
should not be having operators floating around.

2.

I agree with you on point 2 about LOProject. In my previous comment, I had it 
the other way around.

8. 

Yes, flatten is associated with each column of LOGenerate. The columns 
themselves are expression operators and I thought it was better if its an 
attribute of the operator.


Generate is also used for a couple of things:
   i. In Group and Cogroup, the group by columns are wrapped in LOGenerate
   ii. In nested statements in foreach, the expression operators are wrapped in 
LOGenerate.  In the example below, b refers to the column in c. LOGenerate 
wraps up b as distinct expects a relational operator.

{code}
foreach c {a = distinct b; generate b;}
{code}


I am able to compile. I had problems with Exectype.LOCAL (at run time) which 
has been commented out. Apart from that, I am having trouble with instantiating 
functions with PigContext.

> Rework logical plan
> -------------------
>
>                 Key: PIG-158
>                 URL: https://issues.apache.org/jira/browse/PIG-158
>             Project: Pig
>          Issue Type: Sub-task
>          Components: impl
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>         Attachments: logical_operators.patch, logical_operators_rev_1.patch, 
> logical_operators_rev_2.patch, logical_operators_rev_3.patch, 
> parser_changes.patch, ParserErrors.txt, visitorWalker.patch
>
>
> Rework the logical plan in line with 
> http://wiki.apache.org/pig/PigExecutionModel

-- 
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