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

Shravan Matthur Narayanamurthy commented on PIG-157:
----------------------------------------------------

Thanks for reviewing the patch Alan.

1) I will modify the tests and create unit tests. However I'll only be able to 
submit a smaller patch then.

2) I will change that.

3) Sure I will put the TODO FIX.

So please go ahead and take a look at PlanVisitor.java. That is the only class 
in the plan pkg that has changed. But do not commit this. I will submit a new 
patch with the suggested changes. If you are ok with PlanVisitor, then you can 
commit this patch instead.

> Add types and rework execution pipeline
> ---------------------------------------
>
>                 Key: PIG-157
>                 URL: https://issues.apache.org/jira/browse/PIG-157
>             Project: Pig
>          Issue Type: New Feature
>          Components: impl
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>         Attachments: Core.patch.zip, patcher.sh
>
>
> This is the tracking bug for the work to add types to pig and rework the 
> execution pipeline.  Individual components of this work are covered in 
> subtasks.
> Functional and design specs for this work are:
> http://wiki.apache.org/pig/PigTypesFunctionalSpec
> http://wiki.apache.org/pig/PigTypesDesign
> http://wiki.apache.org/pig/PigExecutionModel
> This work is being done on the branch types, since it is large and 
> disruptive, and we want to be able to do incremental checkins without causing 
> issues for the trunk.

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