Uploaded image for project: 'Hadoop Map/Reduce'
  1. Hadoop Map/Reduce
  2. MAPREDUCE-647

Update the DistCp forrest doc to make it consistent with the latest changes (5472, 5620, 5762, 5826)

        Details

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

          Description

          New features have been added to DistCp and the documentation must be updated.

          1. Text File
            MAPREDUCE-647.1.patch
            2 kB
            Rodrigo Schmidt
          2. Text File
            MAPREDUCE-647.patch
            2 kB
            Rodrigo Schmidt

            Issue Links

            depends upon

            Bug - A problem which impairs or prevents the functions of the product. HADOOP-5472 Distcp does not support globbing of input paths

            • Major - Major loss of function.
            • Closed

            Bug - A problem which impairs or prevents the functions of the product. HADOOP-5762 distcp does not copy empty directories

            • Major - Major loss of function.
            • Closed

            Bug - A problem which impairs or prevents the functions of the product. MAPREDUCE-642 distcp could have an option to preserve the full source path

            • Major - Major loss of function.
            • Closed

            Improvement - An improvement or enhancement to an existing feature or task. HADOOP-5620 discp can preserve modification times of files

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              I completely forgot about this.

              Show
              rschmidt Rodrigo Schmidt added a comment - I completely forgot about this.
              Hide
              Permalink
              hadoopqa Hadoop QA added a comment -

              -1 overall. Here are the results of testing the latest attachment
              http://issues.apache.org/jira/secure/attachment/12446701/MAPREDUCE-647.patch
              against trunk revision 952548.

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

              +1 core tests. The patch passed core unit tests.

              -1 contrib tests. The patch failed contrib unit tests.

              Test results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/testReport/
              Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
              Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/artifact/trunk/build/test/checkstyle-errors.html
              Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/console

              This message is automatically generated.

              Show
              hadoopqa Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12446701/MAPREDUCE-647.patch against trunk revision 952548. +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. +1 core tests. The patch passed core unit tests. -1 contrib tests. The patch failed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/232/console This message is automatically generated.
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              This patch contains only some minor documentation changes. I don't think a unit test is really necessary.

              Show
              rschmidt Rodrigo Schmidt added a comment - This patch contains only some minor documentation changes. I don't think a unit test is really necessary.
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              Nicholas, would you mind reviewing this patch?

              Show
              rschmidt Rodrigo Schmidt added a comment - Nicholas, would you mind reviewing this patch?
              Hide
              Permalink
              szetszwo Tsz Wo Nicholas Sze added a comment -

              Sure, I probably could review it today or tomorrow.

              Show
              szetszwo Tsz Wo Nicholas Sze added a comment - Sure, I probably could review it today or tomorrow.
              Hide
              Permalink
              szetszwo Tsz Wo Nicholas Sze added a comment -

              Looks good.

              • do you mean "common ancestor" for "common parent"?
              Show
              szetszwo Tsz Wo Nicholas Sze added a comment - Looks good. do you mean "common ancestor" for "common parent"?
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              Good catch! I'll submit a new patch tonight.

              Show
              rschmidt Rodrigo Schmidt added a comment - Good catch! I'll submit a new patch tonight.
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              New patch.

              Show
              rschmidt Rodrigo Schmidt added a comment - New patch.
              Hide
              Permalink
              hadoopqa Hadoop QA added a comment -

              -1 overall. Here are the results of testing the latest attachment
              http://issues.apache.org/jira/secure/attachment/12447206/MAPREDUCE-647.1.patch
              against trunk revision 955130.

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

              -1 core tests. The patch failed core unit tests.

              -1 contrib tests. The patch failed contrib unit tests.

              Test results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/testReport/
              Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
              Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/artifact/trunk/build/test/checkstyle-errors.html
              Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/console

              This message is automatically generated.

              Show
              hadoopqa Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12447206/MAPREDUCE-647.1.patch against trunk revision 955130. +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. -1 core tests. The patch failed core unit tests. -1 contrib tests. The patch failed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/574/console This message is automatically generated.
              Hide
              Permalink
              rschmidt Rodrigo Schmidt added a comment -

              Nicolas, I made the fix you suggested. Does it look good now?

              Thanks!

              Show
              rschmidt Rodrigo Schmidt added a comment - Nicolas, I made the fix you suggested. Does it look good now? Thanks!
              Hide
              Permalink
              szetszwo Tsz Wo Nicholas Sze added a comment -

              +1 yes, it looks good.

              Show
              szetszwo Tsz Wo Nicholas Sze added a comment - +1 yes, it looks good.
              Hide
              Permalink
              szetszwo Tsz Wo Nicholas Sze added a comment -

              I have committed this. Thanks, Rodrigo!

              Show
              szetszwo Tsz Wo Nicholas Sze added a comment - I have committed this. Thanks, Rodrigo!

                People

                • Assignee:
                  rschmidt Rodrigo Schmidt
                  Reporter:
                  rschmidt Rodrigo Schmidt
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development