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

Deprecate org.apache.hadoop.mapred.lib.MultithreadedMapRunner

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 0.20.1
    • None
    • None
    • Reviewed

    Description

      Deprecate org.apache.hadoop.mapred.lib.MultithreadedMapRunner to use org.apache.hadoop.mapreduce.lib.MultithreadedMapRunner

      Attachments

        1. patch-465.txt
          6 kB
          Amareshwari Sriramadasu
        2. patch-465-0.20.txt
          6 kB
          Amareshwari Sriramadasu
        3. patch-6023.txt
          8 kB
          Amareshwari Sriramadasu

        Activity

          Patch deprecates org.apache.hadoop.mapred.lib.MultithreadedMapRunner to use org.apache.hadoop.mapreduce.lib.map.MultithreadedMapper.
          Fixed minor bugs in MultithreadedMapper. Also added testcase for MultithreadedMapper.

          amareshwari Amareshwari Sriramadasu added a comment - Patch deprecates org.apache.hadoop.mapred.lib.MultithreadedMapRunner to use org.apache.hadoop.mapreduce.lib.map.MultithreadedMapper. Fixed minor bugs in MultithreadedMapper. Also added testcase for MultithreadedMapper.

          test-format

               [exec]
               [exec] +1 overall.
               [exec]
               [exec]     +1 @author.  The patch does not contain any @author tags.
               [exec]
               [exec]     +1 tests included.  The patch appears to include 3 new or modified tests.
               [exec]
               [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
               [exec]
               [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
               [exec]
               [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
               [exec]
               [exec]     +1 Eclipse classpath. The patch retains Eclipse classpath integrity.
               [exec]
               [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.
               [exec]
          
          amareshwari Amareshwari Sriramadasu added a comment - test-format [exec] [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 3 new or modified tests. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 Eclipse classpath. The patch retains Eclipse classpath integrity. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. [exec]

          ant test passed on my machine

          amareshwari Amareshwari Sriramadasu added a comment - ant test passed on my machine

          Attaching patch updated with trunk. Optimized the testcase.

          amareshwari Amareshwari Sriramadasu added a comment - Attaching patch updated with trunk. Optimized the testcase.

          test-patch result:

               [exec]
               [exec] +1 overall.
               [exec]
               [exec]     +1 @author.  The patch does not contain any @author tags.
               [exec]
               [exec]     +1 tests included.  The patch appears to include 3 new or modified tests.
               [exec]
               [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
               [exec]
               [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
               [exec]
               [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
               [exec]
               [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.
               [exec]
               [exec]
          

          TestMultithreadedMapper ran on my machine

          amareshwari Amareshwari Sriramadasu added a comment - test-patch result: [exec] [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 3 new or modified tests. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. [exec] [exec] TestMultithreadedMapper ran on my machine
          sharadag Sharad Agarwal added a comment -

          I committed this. Thanks Amareshwari.

          sharadag Sharad Agarwal added a comment - I committed this. Thanks Amareshwari.
          hudson Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #15 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/15/)

          hudson Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #15 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/15/ )

          Patch for 0.20.1 without the deprecation.
          TestMultithreadedMapper passed on 0.20.1 as well

          amareshwari Amareshwari Sriramadasu added a comment - Patch for 0.20.1 without the deprecation. TestMultithreadedMapper passed on 0.20.1 as well
          sharadag Sharad Agarwal added a comment -

          Committed the bug fix to branch 0.20 as well.

          sharadag Sharad Agarwal added a comment - Committed the bug fix to branch 0.20 as well.

          People

            amareshwari Amareshwari Sriramadasu
            amareshwari Amareshwari Sriramadasu
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: