You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Benjamin Francisoud (JIRA)" <ji...@apache.org> on 2008/02/20 10:52:43 UTC

[jira] Issue Comment Edited: (PIG-98) grunt should show full exception stack

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

francisoud edited comment on PIG-98 at 2/20/08 1:52 AM:
-----------------------------------------------------------------

I think this one has been fix in PIG-83

Now the code looks like [GruntParser.java|http://svn.apache.org/repos/asf/incubator/pig/trunk/src/org/apache/pig/tools/grunt/GruntParser.java] :
{code:java}
            catch(Exception e)
            {
                log.error(e.getMessage());
            }
{code}
You should get the full stacktrace when an error occurs.

      was (Author: francisoud):
    I think this one has been fix in PIG-83

Now the code looks like:
{code:java}
            catch(Exception e)
            {
                log.error(e.getMessage());
            }
{code}
You should get the full stacktrace when an error occurs.
  
> grunt should show full exception stack
> --------------------------------------
>
>                 Key: PIG-98
>                 URL: https://issues.apache.org/jira/browse/PIG-98
>             Project: Pig
>          Issue Type: Improvement
>          Components: grunt
>            Reporter: Stefan Groschupf
>            Priority: Minor
>             Fix For: 0.1.0
>
>         Attachments: showStackTrace-20080207.patch
>
>
> I suggest grunt should be more helpful with user errors. I just did one (a stupid one) and it took my too long to figure out the problem, since grunts error message was just not giving me a good hint:
> grunt> A = LOAD '/pigtestData.tsv' USING PigStorage(',') AS (user,age,cat);
> grunt> B = FILTER A BY cat == 'book';
> grunt> dump B;
> For input string: "book"
> Experts will see that I tried to use == instead of eq, however especially new users could get a little confused. 
> I see two chances add Error Number and  descriptive texts (Oracle style) - this quite a lot of work, or for now I suggest to simply dump the full exception text.
> At least for this early stage it would developers and users to find problems faster.

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