[jira] Commented: (PIG-1041) javac warnings: cast, fallthrough, serial

2009-10-23 Thread Olga Natkovich (JIRA)

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

Olga Natkovich commented on PIG-1041:
-

+1 on the patch. In general, I believe there is a way to supress warnings at 
the package level.

> javac warnings: cast, fallthrough, serial
> -
>
> Key: PIG-1041
> URL: https://issues.apache.org/jira/browse/PIG-1041
> Project: Pig
>  Issue Type: Bug
>Affects Versions: 0.4.0
>Reporter: Daniel Dai
>Assignee: Daniel Dai
> Fix For: 0.6.0
>
> Attachments: PIG-1041-1.patch
>
>
> Pig have javac warnings when you build it with the option "-Dall.warnings=1". 
> We need to suppress all of them. This issue is to track the javac warnings in 
> the following categories:
> cast (49)
> fallthrough (1)
> serial (19)
> The number in the parenthesis is the times of occurrence of particular javac 
> warning.

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



[jira] Commented: (PIG-1041) javac warnings: cast, fallthrough, serial

2009-10-22 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on PIG-1041:


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

+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 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 failed core unit tests.

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

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

This message is automatically generated.

> javac warnings: cast, fallthrough, serial
> -
>
> Key: PIG-1041
> URL: https://issues.apache.org/jira/browse/PIG-1041
> Project: Pig
>  Issue Type: Bug
>Affects Versions: 0.4.0
>Reporter: Daniel Dai
>Assignee: Daniel Dai
> Fix For: 0.6.0
>
> Attachments: PIG-1041-1.patch
>
>
> Pig have javac warnings when you build it with the option "-Dall.warnings=1". 
> We need to suppress all of them. This issue is to track the javac warnings in 
> the following categories:
> cast (49)
> fallthrough (1)
> serial (19)
> The number in the parenthesis is the times of occurrence of particular javac 
> warning.

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



[jira] Commented: (PIG-1041) javac warnings: cast, fallthrough, serial

2009-10-22 Thread Daniel Dai (JIRA)

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

Daniel Dai commented on PIG-1041:
-

The plan is: 
1. Suppressing warnings in the code level as much as possible
2. Some "cast" warnings are from the generated code of javacc, which is out of 
our control. As a result, we will suppress all "cast" warnings in build.xml.

> javac warnings: cast, fallthrough, serial
> -
>
> Key: PIG-1041
> URL: https://issues.apache.org/jira/browse/PIG-1041
> Project: Pig
>  Issue Type: Bug
>Affects Versions: 0.4.0
>Reporter: Daniel Dai
>Assignee: Daniel Dai
> Fix For: 0.6.0
>
>
> Pig have javac warnings when you build it with the option "-Dall.warnings=1". 
> We need to suppress all of them. This issue is to track the javac warnings in 
> the following categories:
> cast (49)
> fallthrough (1)
> serial (19)
> The number in the parenthesis is the times of occurrence of particular javac 
> warning.

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