Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-3037

Hudson needs to add src/test for checking javac warnings

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Not A Problem
    • Affects Version/s: hudson
    • Fix Version/s: hudson
    • Component/s: build
    • Labels:
      None

      Description

      I think src/test is not added in javac warnings checker. HADOOP-3031 looks at warnings introduced.
      Hudson needs to add src/test for checking javac warnings

        Activity

        Hide
        nidaley Nigel Daley added a comment -

        Hudson does check src/test (compile-core-test is run as part of the tar target). However, Hudson counts the total number of javac warnings on trunk and the total number with the patch and only complains if the total number of javac warnings with the patch is higher than trunk. This means that if some warnings are removed and some added, this mechanism won't detect the new warnings.

        Show
        nidaley Nigel Daley added a comment - Hudson does check src/test (compile-core-test is run as part of the tar target). However, Hudson counts the total number of javac warnings on trunk and the total number with the patch and only complains if the total number of javac warnings with the patch is higher than trunk. This means that if some warnings are removed and some added, this mechanism won't detect the new warnings.
        Hide
        aw Allen Wittenauer added a comment -

        Warning checks have been there for a while.

        Show
        aw Allen Wittenauer added a comment - Warning checks have been there for a while.

          People

          • Assignee:
            Unassigned
            Reporter:
            amareshwari Amareshwari Sriramadasu
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development