Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.20.2
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      Corrected arithmetic error that made sleep times less than zero.

      Description

      In this test report

      java.io.IOException: Problem starting http server
      	at org.apache.hadoop.http.HttpServer.start(HttpServer.java:521)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.startHttpServer(NameNode.java:369)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.activate(NameNode.java:315)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.initialize(NameNode.java:306)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.<init>(NameNode.java:412)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.<init>(NameNode.java:406)
      	at org.apache.hadoop.hdfs.server.namenode.NameNode.createNameNode(NameNode.java:1235)
      	at org.apache.hadoop.hdfs.MiniDFSCluster.<init>(MiniDFSCluster.java:287)
      	at org.apache.hadoop.hdfs.MiniDFSCluster.<init>(MiniDFSCluster.java:131)
      	at org.apache.hadoop.hdfs.TestSmallBlock.testSmallBlock(TestSmallBlock.java:98)
      	at org.apache.hadoop.hdfs.TestSmallBlock.testSmallBlockSimulatedStorage(TestSmallBlock.java:112)
      Caused by: java.lang.IllegalArgumentException: timeout value is negative
      	at java.lang.Thread.sleep(Native Method)
      	at org.apache.hadoop.http.HttpServer.start(HttpServer.java:484)
      
      1. HADOOP-6428.0.20.patch
        0.9 kB
        Konstantin Boudnik
      2. HADOOP-6428.0.20.patch
        0.0 kB
        Konstantin Boudnik
      3. HADOOP-6428.patch
        0.9 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk #184 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/184/)
          . HttpServer sleeps with negative values. Contributed by Konstantin Boudnik

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk #184 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/184/ ) . HttpServer sleeps with negative values. Contributed by Konstantin Boudnik
          Hide
          Konstantin Boudnik added a comment -

          I've just committed this

          Show
          Konstantin Boudnik added a comment - I've just committed this
          Hide
          Konstantin Boudnik added a comment -
          There appear to be 0 release audit warnings before the patch and 0 release audit warnings after applying the patch.
          
          +1 overall.  
              +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 new tests are needed for this patch.
                                  Also please list what manual steps were performed to verify 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.
          

          Clearly doesn't have new tests.

          Show
          Konstantin Boudnik added a comment - There appear to be 0 release audit warnings before the patch and 0 release audit warnings after applying the patch. +1 overall. +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 new tests are needed for this patch. Also please list what manual steps were performed to verify 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. Clearly doesn't have new tests.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #104 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/104/)
          . HttpServer sleeps with negative values. Contributed by Konstantin Boudnik

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #104 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/104/ ) . HttpServer sleeps with negative values. Contributed by Konstantin Boudnik
          Hide
          Konstantin Boudnik added a comment -

          I also have ran all the common tests and they were successful (should've post it here before).

          Show
          Konstantin Boudnik added a comment - I also have ran all the common tests and they were successful (should've post it here before).
          Hide
          Konstantin Boudnik added a comment -

          Correct .20 patch

          Show
          Konstantin Boudnik added a comment - Correct .20 patch
          Hide
          Tsz Wo Nicholas Sze added a comment -

          > What have you been running to get to this problem? This is a side effect of HDFS-771 and is extremely hard to reproduce.

          I just have seen this problem in a hudson build.

          (HDFS-771 was changed to HADOOP-6386.)

          Show
          Tsz Wo Nicholas Sze added a comment - > What have you been running to get to this problem? This is a side effect of HDFS-771 and is extremely hard to reproduce. I just have seen this problem in a hudson build. ( HDFS-771 was changed to HADOOP-6386 .)
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1 patch looks good.

          Show
          Tsz Wo Nicholas Sze added a comment - +1 patch looks good.
          Hide
          Konstantin Boudnik added a comment -

          0.20 patch

          Show
          Konstantin Boudnik added a comment - 0.20 patch
          Hide
          Konstantin Boudnik added a comment -

          That should do it.

          What have you been running to get to this problem? This is a side effect of HDFS-771 and is extremely hard to reproduce.

          Show
          Konstantin Boudnik added a comment - That should do it. What have you been running to get to this problem? This is a side effect of HDFS-771 and is extremely hard to reproduce.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development