[
https://issues.apache.org/jira/browse/PIG-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12583210#action_12583210
]
Alan Gates commented on PIG-158:
--------------------------------
I've committed logical_operator_rev_3.patch. A few comments on it:
1) This patch was out of date with top of the branch. In the future, please be
sure to do an svn update before generating the patch.
2) I made some changes to LOVisitor. It does not need to be parameterized. It
just needs to extend PlanVisitor<LogicalOperator, LOVisitor>. I changed it
accordingly.
3) We need some unit tests for LOVisitor. Please create some unit tests that
test that visiting works correctly.
With this patch in place, everything under logicalLayer (except in the parser
directory) now compiles.
> 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
>
>
> 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.
