Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21.0
    • Component/s: test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      There are javac warnings in the following tests:

      src/test/hdfs/org/apache/hadoop/hdfs/TestFileCreation.java
      src/test/hdfs/org/apache/hadoop/hdfs/TestSmallBlock.java
      src/test/hdfs/org/apache/hadoop/hdfs/TestFileStatus.java
      src/test/hdfs/org/apache/hadoop/hdfs/TestDFSShellGenericOptions.java
      src/test/hdfs/org/apache/hadoop/hdfs/TestSeekBug.java
      src/test/hdfs/org/apache/hadoop/hdfs/TestDFSStartupVersions.java
      
      1. 5954_20090601.patch
        6 kB
        Tsz Wo Nicholas Sze

        Activity

        Hide
        Tsz Wo Nicholas Sze added a comment -

        5953_20090601.patch: fix javac warnings

        Show
        Tsz Wo Nicholas Sze added a comment - 5953_20090601.patch: fix javac warnings
        Hide
        Tsz Wo Nicholas Sze added a comment -

        5954_20090601.patch: this is the correct file.

        Show
        Tsz Wo Nicholas Sze added a comment - 5954_20090601.patch: this is the correct file.
        Hide
        Jakob Homan added a comment -

        +1. Looks good.

        Show
        Jakob Homan added a comment - +1. Looks good.
        Hide
        Tsz Wo Nicholas Sze added a comment -
             [exec] +1 overall.  
             [exec] 
             [exec]     +1 @author.  The patch does not contain any @author tags.
             [exec] 
             [exec]     +1 tests included.  The patch appears to include 20 new or modified tests.
             [exec] 
             [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
             [exec] 
             [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
             [exec] 
             [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
             [exec] 
             [exec]     +1 Eclipse classpath. The patch retains Eclipse classpath integrity.
             [exec] 
             [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.
        

        Run all tests mentioned in the description locally. Passed all of them.

        Show
        Tsz Wo Nicholas Sze added a comment - [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 20 new or modified tests. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 Eclipse classpath. The patch retains Eclipse classpath integrity. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. Run all tests mentioned in the description locally. Passed all of them.
        Hide
        Tsz Wo Nicholas Sze added a comment -

        I have committed this.

        Show
        Tsz Wo Nicholas Sze added a comment - I have committed this.

          People

          • Assignee:
            Tsz Wo Nicholas Sze
            Reporter:
            Tsz Wo Nicholas Sze
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development