Pig
  1. Pig
  2. PIG-854

The automated build process should publish the diff in compiler warning messages

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 0.3.0
    • Fix Version/s: None
    • Component/s: build
    • Labels:
      None
    • Environment:

      Hudson

      Description

      Currently, the automated build process publishes a report that captures the difference in the number of warning messages due to a patch from that of trunk However, the details of the new warning messages are not listed. For findbugs, a url that contains the details is published. A similar page is required for the compiler warning messages.

      For reference, check out

      https://issues.apache.org/jira/browse/PIG-697?focusedCommentId=12720326&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12720326

      The console output has the following line. Note that the details are stored on the build machine. Its not exported as a web page for users to figure out the details. At least, I was not able to do so. It would be extremely helpful (I would say mandatory) to expose the details. If these details are already present then please point me to the right location.

      [exec] /home/hudson/tools/ant/latest/bin/ant  -Djavac.args=-Xlint -Xmaxwarns 1000 -Declipse.home=/home/nigel/tools/eclipse/latest -Djava5.home=/home/hudson/tools/java/latest1.5 -Dforrest.home=/home/nigel/tools/forrest/latest -DPigPatchProcess= clean tar > /home/hudson/hudson-slave/workspace/Pig-Patch-minerva.apache.org/patchprocess/patchJavacWarnings.txt 2>&1
           [exec] There appear to be 224 javac compiler warnings before the patch and 259 javac compiler warnings after applying the patch.
      

        Activity

        Santhosh Srinivasan created issue -
        Hide
        Nigel Daley added a comment -

        Get the project javac and javadoc warnings to 0 and then this becomes a non-issue. That's what Hadoop is working on.

        Show
        Nigel Daley added a comment - Get the project javac and javadoc warnings to 0 and then this becomes a non-issue. That's what Hadoop is working on.
        Hide
        Santhosh Srinivasan added a comment -

        Thats a nice to have and besides the point. Even if there is one warning, the build system should display that single warning.

        Show
        Santhosh Srinivasan added a comment - Thats a nice to have and besides the point. Even if there is one warning, the build system should display that single warning.
        Hide
        Nigel Daley added a comment -

        I believe this can be done by the Warnings hudson plugin.

        Show
        Nigel Daley added a comment - I believe this can be done by the Warnings hudson plugin.
        Olga Natkovich made changes -
        Field Original Value New Value
        Fix Version/s 0.3.0 [ 12313785 ]

          People

          • Assignee:
            Giridharan Kesavan
            Reporter:
            Santhosh Srinivasan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development