[jira] Commented: (PIG-1041) javac warnings: cast, fallthrough, serial
[ 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
[ 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
[ 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.