Details

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

      Description

      This is for patch tracking of HDFS part of HADOOP-6222

      1. HDFS-832.patch
        9 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Hide
          cos Konstantin Boudnik added a comment -

          This patch is simply copied over from HADOOP-6222 (formerly known as HADOOP-6222.hdfs-part.patch).

          Show
          cos Konstantin Boudnik added a comment - This patch is simply copied over from HADOOP-6222 (formerly known as HADOOP-6222 .hdfs-part.patch).
          Hide
          cos Konstantin Boudnik added a comment -

          HADOOP-6222 has been integrated. So, time to verify this one.

          Show
          cos Konstantin Boudnik added a comment - HADOOP-6222 has been integrated. So, time to verify this one.
          Hide
          cos Konstantin Boudnik added a comment -

          The test-patch process takes too long to kick off. So, I've ran it locally with the following results:

          There appear to be 118 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 appears to include 12 new or modified tests.
              +1 javadoc.  The javadoc tool did not generate any warning messages.
              -1 javac.  The patch appears to cause tar ant target to fail.
              +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.
          

          Aforementioned 118 warnings are actually compilation errors cause by HADOOP-6222 modifications. Clearly, as soon as the patch has been applied all of them were fixed. I also had ran TestHDFSCLI test and all commit tests and everything has passes. So, it seems that the patch is good to go with the commit.

          Show
          cos Konstantin Boudnik added a comment - The test-patch process takes too long to kick off. So, I've ran it locally with the following results: There appear to be 118 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 appears to include 12 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. -1 javac. The patch appears to cause tar ant target to fail. +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. Aforementioned 118 warnings are actually compilation errors cause by HADOOP-6222 modifications. Clearly, as soon as the patch has been applied all of them were fixed. I also had ran TestHDFSCLI test and all commit tests and everything has passes. So, it seems that the patch is good to go with the commit.
          Hide
          cos Konstantin Boudnik added a comment -

          The patch has been reviewed as the part of HADOOP-6222.
          I've just committed this.

          Show
          cos Konstantin Boudnik added a comment - The patch has been reviewed as the part of HADOOP-6222 . I've just committed this.
          Hide
          hudson Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #148 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/148/)
          . HDFS side of HADOOP-6222. Contributed by Konstantin Boudnik.

          Show
          hudson Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #148 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/148/ ) . HDFS side of HADOOP-6222 . Contributed by Konstantin Boudnik.
          Hide
          hudson Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #172 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk/172/)
          . HDFS side of HADOOP-6222. Contributed by Konstantin Boudnik.

          Show
          hudson Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #172 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk/172/ ) . HDFS side of HADOOP-6222 . Contributed by Konstantin Boudnik.
          Hide
          hudson Hudson added a comment -

          Integrated in Hdfs-Patch-h5.grid.sp2.yahoo.net #148 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h5.grid.sp2.yahoo.net/148/)

          Show
          hudson Hudson added a comment - Integrated in Hdfs-Patch-h5.grid.sp2.yahoo.net #148 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h5.grid.sp2.yahoo.net/148/ )
          Hide
          hudson Hudson added a comment -

          Integrated in Hdfs-Patch-h2.grid.sp2.yahoo.net #94 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h2.grid.sp2.yahoo.net/94/)

          Show
          hudson Hudson added a comment - Integrated in Hdfs-Patch-h2.grid.sp2.yahoo.net #94 (See http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h2.grid.sp2.yahoo.net/94/ )

            People

            • Assignee:
              cos Konstantin Boudnik
              Reporter:
              cos Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development