Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.21.0
    • test
    • None
    • Reviewed

    Description

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

      Attachments

        1. HDFS-832.patch
          9 kB
          Konstantin I Boudnik

        Issue Links

          Activity

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

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

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

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

            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.

            cos Konstantin I 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.

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

            cos Konstantin I Boudnik added a comment - The patch has been reviewed as the part of HADOOP-6222 . I've just committed this.
            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.

            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.
            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.

            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.
            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/)

            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/ )
            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/)

            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

              cos Konstantin I Boudnik
              cos Konstantin I Boudnik
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: