[
https://issues.apache.org/jira/browse/PIG-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12580773#action_12580773
]
Pi Song commented on PIG-158:
-----------------------------
It looks good.
Comments:-
1) How do you maintain relations between LOs and dependencies in the abstract
graph layer (OperatorPlan) in the same time?
2) mSortFunc in LOSort is declared as LOUserFunc. If I use a built-in sort, it
will still be LOUserFunc. When we do optimization built-in funcs and UDFs are
treated differently so I think here you should declare a more generic type.
3) LOCogroup seems only define inputs, how to specify columns to be grouped?
Will have a look more next week. Thanks Santhosh !!!
> 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
>
>
> 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.
