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

Shravan Matthur Narayanamurthy updated PIG-161:
-----------------------------------------------

    Attachment: exprplan.patch

The physical side has a separate notion of ExprPlan & a visitor for the same. 
It was fine till now because the role of each operator was clear. However with 
the solution for the foreach has the inner plans which can have expression 
operators interspersed. Now for these operators the visit method is called with 
a PhyPlanVisitor whereas them being expr operators expect a ExprPlan visitor. 
So in this patch I have now removed ExprPlan, ExprPlanVisitor, made 
PhysicalOperator and PhysicalPlan untyped. This closely resembles the logical 
side now and simplifies the visitor logic.

> Rework physical plan
> --------------------
>
>                 Key: PIG-161
>                 URL: https://issues.apache.org/jira/browse/PIG-161
>             Project: Pig
>          Issue Type: Sub-task
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>         Attachments: arithmeticOperators.patch, BinCondAndNegative.patch, 
> CastAndMapLookUp.patch, exprplan.patch, incr2.patch, incr3.patch, 
> incr4.patch, incr5.patch, logToPhyTranslator.patch, missingOps.patch, 
> MRCompilerTests_PlansAndOutputs.txt, Phy_AbsClass.patch, physicalOps.patch, 
> physicalOps.patch, physicalOps.patch, physicalOps.patch, 
> physicalOps_latest.patch, POCast.patch, POCast.patch, podistinct.patch, 
> pogenerate.patch, pogenerate.patch, pogenerate.patch, posort.patch, 
> POUserFuncCorrection.patch, 
> TEST-org.apache.pig.test.TestLocalJobSubmission.txt, 
> TEST-org.apache.pig.test.TestLogToPhyCompiler.txt, 
> TEST-org.apache.pig.test.TestLogToPhyCompiler.txt, 
> TEST-org.apache.pig.test.TestMapReduce.txt, 
> TEST-org.apache.pig.test.TestTypeCheckingValidator.txt, 
> TEST-org.apache.pig.test.TestUnion.txt, translator.patch, translator.patch, 
> translator.patch, translator.patch
>
>
> This bug tracks work to rework all of the physical operators as described in 
> http://wiki.apache.org/pig/PigTypesFunctionalSpec

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