[
https://issues.apache.org/jira/browse/PIG-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12582385#action_12582385
]
Santhosh Srinivasan commented on PIG-158:
-----------------------------------------
I verified the svn diff visually.
1. I did not find any code duplication.
2. LOEval has been removed. It is also removed from the build.xml. I am not
sure why it keeps showing up in the patch that you applied.
3. The build.xml should include the new source LOProject.java.
I went ahead and updated my source tree under types. The apache
commoins-collection.jar has been removed and prevented me from compiling
OperatorPlan.java (import org.apache.commons.collections.map.MultiValueMap;) I
copied a saved version of the jar file and was able to compile.
I will attach the new patch. Could you please check if this patch works for you?
Thanks for the inputs.
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, logical_operators_rev_1.patch,
> logical_operators_rev_2.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.
