Details

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

      Description

      Gridmix forrest documentation

      1. MAPREDUCE-1931.patch
        31 kB
        Ranjit Mathew
      2. gridmix-document.patch
        13 kB
        rahul k singh
      3. gridmix.pdf
        24 kB
        rahul k singh

        Activity

        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/)

        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/ )
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk-Commit #554 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/554/)

        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #554 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/554/ )
        Hide
        Vinod Kumar Vavilapalli added a comment -

        I just committed to trunk and branch-0.22. Thanks Ranjit!

        Show
        Vinod Kumar Vavilapalli added a comment - I just committed to trunk and branch-0.22. Thanks Ranjit!
        Hide
        Ravi Gummadi added a comment -

        Patch looks good.
        +1

        Show
        Ravi Gummadi added a comment - Patch looks good. +1
        Hide
        Ranjit Mathew added a comment -

        Here is a revamped patch that builds on Rahul's patch to reflect the current trunk.

        Show
        Ranjit Mathew added a comment - Here is a revamped patch that builds on Rahul's patch to reflect the current trunk.
        Hide
        Ranjit Mathew added a comment -

        I meant "bin/hadoop jar". :-/

        Hong, I agree that the documentation should reflect "as-packaged" paths rather than "in-source" paths, as the documentation
        is for end-users of Hadoop, not Hadoop developers.

        I favour explicitly describing the parameters from MAPREDUCE-1936 in the documentation. Do you want to update the patch?

        Show
        Ranjit Mathew added a comment - I meant "bin/hadoop jar". :-/ Hong, I agree that the documentation should reflect "as-packaged" paths rather than "in-source" paths, as the documentation is for end-users of Hadoop, not Hadoop developers. I favour explicitly describing the parameters from MAPREDUCE-1936 in the documentation. Do you want to update the patch?
        Hide
        Hong Tang added a comment -
        • The description of the locations of jars is inaccurate if people use prebuilt hadoop package. In such cases, the jars should be under HADOOP_HOME and HADOOP_HOME/contrib/gridmix/.
        • A few new configuration parameters are added as part of MAPREDUCE-1936. Should we document those too? If you run Gridmix3 without any parameters, you would see these options. Alternatively, we can just say in the forrest documentation that information about additional configuration parameters are available through Gridmix command line help.
        Show
        Hong Tang added a comment - The description of the locations of jars is inaccurate if people use prebuilt hadoop package. In such cases, the jars should be under HADOOP_HOME and HADOOP_HOME/contrib/gridmix/. A few new configuration parameters are added as part of MAPREDUCE-1936 . Should we document those too? If you run Gridmix3 without any parameters, you would see these options. Alternatively, we can just say in the forrest documentation that information about additional configuration parameters are available through Gridmix command line help.
        Hide
        Ranjit Mathew added a comment -

        This mostly looks fine, except that I don't favour removing the "bin/mapred jar" bit from the command-invocations.

        Show
        Ranjit Mathew added a comment - This mostly looks fine, except that I don't favour removing the "bin/mapred jar" bit from the command-invocations.
        Hide
        rahul k singh added a comment -

        Attaching patch for 20.1xx release.

        Show
        rahul k singh added a comment - Attaching patch for 20.1xx release.

          People

          • Assignee:
            Ranjit Mathew
            Reporter:
            rahul k singh
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development