Grokbase Groups Pig dev May 2010
FAQ
Suggest to add clear command in grunt to clear the relation variable
--------------------------------------------------------------------

Key: PIG-1423
URL: https://issues.apache.org/jira/browse/PIG-1423
Project: Pig
Issue Type: New Feature
Components: grunt
Affects Versions: 0.8.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang
Fix For: 0.8.0


The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.

students = load 'a.txt';
....
student = load 'b.txt';
result = foreach {color:red}students{color} generate $0;


The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.




--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Search Discussions

  • Jeff Zhang (JIRA) at May 20, 2010 at 6:24 am
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Jeff Zhang updated PIG-1423:
    ----------------------------

    Attachment: PIG-1423.patch
    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Jeff Zhang (JIRA) at May 20, 2010 at 6:24 am
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Jeff Zhang updated PIG-1423:
    ----------------------------

    Status: Patch Available (was: Open)
    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Hadoop QA (JIRA) at May 27, 2010 at 10:28 pm
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12872664#action_12872664 ]

    Hadoop QA commented on PIG-1423:
    --------------------------------

    -1 overall. Here are the results of testing the latest attachment
    http://issues.apache.org/jira/secure/attachment/12445029/PIG-1423.patch
    against trunk revision 948526.

    +1 @author. The patch does not contain any @author tags.

    +1 tests included. The patch appears to include 3 new or modified tests.

    -1 javadoc. The javadoc tool appears to have generated 1 warning messages.

    +1 javac. The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs. The patch does not introduce any new Findbugs warnings.

    +1 release audit. The applied patch does not increase the total number of release audit warnings.

    +1 core tests. The patch passed core unit tests.

    +1 contrib tests. The patch passed contrib unit tests.

    Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h1.grid.sp2.yahoo.net/9/testReport/
    Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h1.grid.sp2.yahoo.net/9/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
    Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h1.grid.sp2.yahoo.net/9/console

    This message is automatically generated.
    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Olga Natkovich (JIRA) at Jul 27, 2010 at 11:49 pm
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12892992#action_12892992 ]

    Olga Natkovich commented on PIG-1423:
    -------------------------------------

    I am not sure I like the idea of unsetting variables as a grant command. I think it can cause more confusion and generate more problems then it would help to solve. I think the right way to solve this problem is to define scope for variables as part of our turing complete effort.

    If we have an agreement on this, I would like to unlinik it from 0.8.0 release.


    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Jeff Zhang (JIRA) at Jul 28, 2010 at 1:00 am
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12893016#action_12893016 ]

    Jeff Zhang commented on PIG-1423:
    ---------------------------------

    Yes, Olga, I agree with you
    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.
  • Olga Natkovich (JIRA) at Jul 28, 2010 at 4:23 pm
    [ https://issues.apache.org/jira/browse/PIG-1423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Olga Natkovich updated PIG-1423:
    --------------------------------

    Status: Resolved (was: Patch Available)
    Resolution: Won't Fix

    This will be resolved as part of turing complete work
    Suggest to add clear command in grunt to clear the relation variable
    --------------------------------------------------------------------

    Key: PIG-1423
    URL: https://issues.apache.org/jira/browse/PIG-1423
    Project: Pig
    Issue Type: New Feature
    Components: grunt
    Affects Versions: 0.8.0
    Reporter: Jeff Zhang
    Assignee: Jeff Zhang
    Fix For: 0.8.0

    Attachments: PIG-1423.patch


    The relation name in pig script can been considered as variable of programming language. One risk of the current grunt is that user may use the previous defined relation name by typo after a long period of work on grunt. And it is difficult for users to track this problem. E.g. the following red "students" is not the user intend to use, but here grunt won't throw any error meesage.
    students = load 'a.txt';
    ....
    student = load 'b.txt';
    result = foreach {color:red}students{color} generate $0;
    The clear command is to clear the variable defined before, then if users use the relation name defined before, grunt will throw error message. And this command also will be useful to let user reuse the relation names especially when he's doing lots of experiments for one specific task.
    --
    This message is automatically generated by JIRA.
    -
    You can reply to this email to add a comment to the issue online.

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupdev @
categoriespig, hadoop
postedMay 20, '10 at 6:10a
activeJul 28, '10 at 4:23p
posts7
users1
websitepig.apache.org

1 user in discussion

Olga Natkovich (JIRA): 7 posts

People

Translate

site design / logo © 2022 Grokbase