Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-889

binary communication formats added to Streaming by HADOOP-1722 should be documented

    Details

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

      Description

      binary communication formats added to Streaming by HADOOP-1722 should be documented in forrest

      1. MAPREDUCE-889.patch
        2 kB
        Klaas Bosteels
      2. MAPREDUCE-889.patch
        3 kB
        Klaas Bosteels

        Activity

        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk #285 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/285/)
        MAPREDUCE-889. binary communication formats added to Streaming by HADOOP-1722 should be documented. Contributed by Klaas Bosteels.

        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #285 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/285/ ) MAPREDUCE-889 . binary communication formats added to Streaming by HADOOP-1722 should be documented. Contributed by Klaas Bosteels.
        Hide
        Tom White added a comment -

        I've just committed this. Thanks Klaas!

        Show
        Tom White added a comment - I've just committed this. Thanks Klaas!
        Hide
        Klaas Bosteels added a comment -

        Obviously, the unit test failures are not related.

        Show
        Klaas Bosteels added a comment - Obviously, the unit test failures are not related.
        Hide
        Hadoop QA added a comment -

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

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

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12439036/MAPREDUCE-889.patch against trunk revision 927488. +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 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/61/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/61/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/61/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Mapreduce-Patch-h4.grid.sp2.yahoo.net/61/console This message is automatically generated.
        Hide
        Amareshwari Sriramadasu added a comment -

        I ran ant docs with the patch; and it was successful.

        Show
        Amareshwari Sriramadasu added a comment - I ran ant docs with the patch; and it was successful.
        Hide
        Amareshwari Sriramadasu added a comment -

        Sorry I missed the last update. Thanks Klaas for updating the patch. Changes look fine to me.

        Show
        Amareshwari Sriramadasu added a comment - Sorry I missed the last update. Thanks Klaas for updating the patch. Changes look fine to me.
        Hide
        Klaas Bosteels added a comment -

        Looks like there are no more comments, so I'm going to send this through hudson...

        Show
        Klaas Bosteels added a comment - Looks like there are no more comments, so I'm going to send this through hudson...
        Hide
        Klaas Bosteels added a comment -

        Thanks Amareshwari, I've added the link. Any other comments?

        Show
        Klaas Bosteels added a comment - Thanks Amareshwari, I've added the link. Any other comments?
        Hide
        Amareshwari Sriramadasu added a comment -

        For the following change can you add the link to api documentation.

        +<li><code>typedbytes</code>: The "typed bytes" format as described in the API documentation for the package <code>org.apache.hadoop.typedbytes</code>.</li>
        

        You can add the external ref in site.xml, and link it from the streaming documentation through href.
        For example, see :

        <a href="ext:api/org/apache/hadoop/mapred/lib/keyfieldbasedpartitioner">KeyFieldBasedPartitioner</a>, 
        
        Show
        Amareshwari Sriramadasu added a comment - For the following change can you add the link to api documentation. +<li><code>typedbytes</code>: The "typed bytes" format as described in the API documentation for the package <code>org.apache.hadoop.typedbytes</code>.</li> You can add the external ref in site.xml, and link it from the streaming documentation through href. For example, see : <a href= "ext:api/org/apache/hadoop/mapred/lib/keyfieldbasedpartitioner" >KeyFieldBasedPartitioner</a>,
        Hide
        Klaas Bosteels added a comment -

        Does the attached patch provide the documentation you had in mind, Amareshwari?

        Show
        Klaas Bosteels added a comment - Does the attached patch provide the documentation you had in mind, Amareshwari?

          People

          • Assignee:
            Klaas Bosteels
            Reporter:
            Amareshwari Sriramadasu
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development