Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-858

clean up smallJobsBenchmark and move to src/test/org/apache/hadoop/mapred

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 0.9.2
    • 0.12.0
    • build
    • None

    Description

      Finish off the missing piece of HADOOP-371 (move src/contrib/smallJobsBenchmark to src/test/org/apache/hadoop/mapred) and cleanup or consolidate documentation, scripts and dead code.

      Attachments

        1. HADOOP-858.patch
          38 kB
          Nigel Daley
        2. HADOOP-858-2.patch
          38 kB
          Nigel Daley

        Activity

          nidaley Nigel Daley added a comment -

          The attached patch moves the smallJobsBenchmark code from contrib into src/test/org/apache/hadoop/mapred/MRBench.java. Since the patch removes some files, it should be applied with
          patch -E -p0
          The code has been largely refactored. The command line args have also changed. See the usage message or the main method javadoc.

          nidaley Nigel Daley added a comment - The attached patch moves the smallJobsBenchmark code from contrib into src/test/org/apache/hadoop/mapred/MRBench.java. Since the patch removes some files, it should be applied with patch -E -p0 The code has been largely refactored. The command line args have also changed. See the usage message or the main method javadoc.
          nidaley Nigel Daley added a comment -

          fixed a small bug

          nidaley Nigel Daley added a comment - fixed a small bug
          hadoopqa Hadoop QA added a comment -

          +1, because http://issues.apache.org/jira/secure/attachment/12350482/HADOOP-858-2.patch applied and successfully tested against trunk revision r503864.

          hadoopqa Hadoop QA added a comment - +1, because http://issues.apache.org/jira/secure/attachment/12350482/HADOOP-858-2.patch applied and successfully tested against trunk revision r503864.
          cutting Doug Cutting added a comment -

          I just committed this. Thanks, Nigel!

          cutting Doug Cutting added a comment - I just committed this. Thanks, Nigel!

          People

            nidaley Nigel Daley
            nidaley Nigel Daley
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: