hadoop-pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Gates (JIRA)" <j...@apache.org>
Subject [jira] Commented: (PIG-1464) Should clean the Graph when register another Pig Script
Date Fri, 25 Jun 2010 18:39:54 GMT

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

Alan Gates commented on PIG-1464:
---------------------------------

I agree this is weird from an interface viewpoint.  But I have a couple of concerns in changing
it.  One, it isn't backward compatible.  After the mess we drug users through in 0.7 we're
really trying not to change anything for 0.8.  The other concern is that it gives users a
very hacky way to build Pig Latin modules and use them together.  We'd like to come up with
a clean way to do this (see http://wiki.apache.org/pig/TuringCompletePig )  But until then
I'm wondering if we should leave this there.

> Should clean the Graph when register another Pig Script
> -------------------------------------------------------
>
>                 Key: PIG-1464
>                 URL: https://issues.apache.org/jira/browse/PIG-1464
>             Project: Pig
>          Issue Type: Bug
>          Components: grunt
>    Affects Versions: 0.8.0
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>             Fix For: 0.8.0
>
>         Attachments: PIG_1463.patch
>
>
> In the current implementation, the variable names in pig script are all global variable.
This make one pig script know the variable in other scripts. In my opinion, this is not right.
Every relation name in pig script should be local variable, otherwise it will bring in unexpected
result.  This issue relates to PIG-1423
> E.g there are two pig script as follows:
> Test_1.pig
> {code}
> a = load 'data/b.txt' ;
> {code}
> Test_2.pig
> {code}
> b = foreach a generate $0;   // "a" is recognized by Grunt although it is in Test_1.pig
> {code}
> And the following execute normally, do not throw any exception
> {code}
> PigServer pig=new PigServer(ExecType.Local);
> pig.registerScript("Test_1.pig");
> pig.registerScript("Test_2.pig");
> {code}

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