hadoop-pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Srinivasan (JIRA)" <j...@apache.org>
Subject [jira] Commented: (PIG-161) Rework physical plan
Date Sat, 24 May 2008 00:46:00 GMT

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

Santhosh Srinivasan commented on PIG-161:
-----------------------------------------

Shubham,

For LOUserFunc, the function arguments are part of LOUserFunc and are not included in the
plan, i.e., when you visit LOUserFunc, you need to visit the function arguments using the
getArguments method.

{code}
    protected void visit(LOUserFunc func) throws VisitorException {
        // Visit each of the arguments
        Iterator<ExpressionOperator> i = func.getArguments().iterator();
        while (i.hasNext()) {
            i.next().visit(this);
        }
    }
{code}

For the store error, I have to fix the parser code that returns a logical plan for LOStore

> 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,
incr2.patch, incr3.patch, incr4.patch, incr5.patch, logToPhyTranslator.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, 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.


Mime
View raw message