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

Forrest documentation should be updated to reflect the changes in MAPREDUCE-856

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.21.0, 0.22.0
    • 0.21.0
    • documentation
    • None
    • Reviewed
    • Hide
      Updated forrest documentation to reflect the changes to make localized files from DistributedCache have right access-control on TaskTrackers(MAPREDUCE-856).
      Show
      Updated forrest documentation to reflect the changes to make localized files from DistributedCache have right access-control on TaskTrackers( MAPREDUCE-856 ).

    Description

      The directory structure under mapred-local-dir is changed in MAPREDUCE-856. This needs changes to the forrest documentation.

      Attachments

        1. MAPREDUCE-1610-20.patch
          6 kB
          Hemanth Yamijala
        2. MR1610.patch
          7 kB
          Ravi Gummadi
        3. MR1610.v1.patch
          8 kB
          Ravi Gummadi

        Activity

          Attaching a patch on behalf of Ravi. This is for earlier versions of Hadoop and not for commit here.

          yhemanth Hemanth Yamijala added a comment - Attaching a patch on behalf of Ravi. This is for earlier versions of Hadoop and not for commit here.
          ravidotg Ravi Gummadi added a comment -

          Attaching patch for trunk.

          ravidotg Ravi Gummadi added a comment - Attaching patch for trunk.

          Patch mostly looks good. Only one minor comment - the directory structure under the IsolationRunner section is still not updated.

          vinodkv Vinod Kumar Vavilapalli added a comment - Patch mostly looks good. Only one minor comment - the directory structure under the IsolationRunner section is still not updated.
          ravidotg Ravi Gummadi added a comment -

          Attaching new patch for trunk with review comment incorporated.

          ravidotg Ravi Gummadi added a comment - Attaching new patch for trunk with review comment incorporated.

          Patch looks good. Running it through Hudson.

          vinodkv Vinod Kumar Vavilapalli added a comment - Patch looks good. Running it through Hudson.
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12444089/MR1610.v1.patch
          against trunk revision 941564.

          +1 @author. The patch does not contain any @author tags.

          +0 tests included. The patch appears to be a documentation patch that doesn't require tests.

          +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/535/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/console

          This message is automatically generated.

          hadoopqa Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12444089/MR1610.v1.patch against trunk revision 941564. +1 @author. The patch does not contain any @author tags. +0 tests included. The patch appears to be a documentation patch that doesn't require tests. +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/535/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h6.grid.sp2.yahoo.net/535/console This message is automatically generated.
          ravidotg Ravi Gummadi added a comment -

          Patch changes only documentation. Tests failures are not related to this patch.

          ravidotg Ravi Gummadi added a comment - Patch changes only documentation. Tests failures are not related to this patch.

          I just committed this to branch 0.21 and trunk. Thanks Ravi!

          vinodkv Vinod Kumar Vavilapalli added a comment - I just committed this to branch 0.21 and trunk. Thanks Ravi!

          People

            ravidotg Ravi Gummadi
            ravidotg Ravi Gummadi
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: