[ 
https://issues.apache.org/jira/browse/PIG-158?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alan Gates updated PIG-158:
---------------------------

    Attachment: visitorWalker.patch

Attached is a patch with proposed interface changes to visitors to match my 
suggestion in comment 
http://issues.apache.org/jira/browse/PIG-158?focusedCommentId=12584744#action_12584744

I apologize for all the extraneous changes in there, as I also changed 
ParseException to VisitorException and that added 2-4 lines from a bunch of 
files.  The relevant files to look at are PlanVisitor, PlanWalker, 
DependencyOrderWalker, DepthFirstWalker, and LOVisitor.  Changes to the 
existing visitors is quite small.  All they need to do is instantiate the 
appropriate type of walker at construction time.

Hopefully this will give us an idea of how the visitors would look and work in 
that propsoal.

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