hadoop-pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <j...@apache.org>
Subject [jira] Commented: (PIG-728) All backend error messages must be logged to preserve the original error messages
Date Thu, 16 Jul 2009 01:15:14 GMT

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

Hadoop QA commented on PIG-728:
-------------------------------

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

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

    -1 tests included.  The patch doesn't appear to include any new or modified tests.
                        Please justify why no tests are needed for this patch.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

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

    -1 findbugs.  The patch appears to introduce 1 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-minerva.apache.org/127/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-minerva.apache.org/127/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-minerva.apache.org/127/console

This message is automatically generated.

> All backend error messages must be logged to preserve the original error messages
> ---------------------------------------------------------------------------------
>
>                 Key: PIG-728
>                 URL: https://issues.apache.org/jira/browse/PIG-728
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: 0.3.0
>            Reporter: Santhosh Srinivasan
>            Assignee: Santhosh Srinivasan
>            Priority: Minor
>             Fix For: 0.4.0
>
>         Attachments: PIG-728.patch
>
>
> The current error handling framework logs backend error messages only when Pig is not
able to parse the error message. Instead, Pig should log the backend error message irrespective
of Pig's ability to parse backend error messages. On a side note, the use of instantiateFuncFromSpec
in Launcher.java is not consistent and should avoid the use of class_name + "(" + string_constructor_args
+ ")".

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