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

Reduce cannot use more than 2G memory for the final merge

    Details

    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      In the org.apache.hadoop.mapreduce.task.reduce.MergeManagerImpl.java file, in the finalMerge method:

      int maxInMemReduce = (int)Math.min(
      Runtime.getRuntime().maxMemory() * maxRedPer, Integer.MAX_VALUE);

      This means no matter how much memory user has, reducer will not retain more than 2G data in memory before the reduce phase starts.

      1. MAPREDUCE-5649.001.patch
        6 kB
        Gera Shegalov
      2. MAPREDUCE-5649.002.patch
        6 kB
        Gera Shegalov
      3. MAPREDUCE-5649.003.patch
        6 kB
        Gera Shegalov

        Issue Links

          Activity

          Hide
          milindb Milind Bhandarkar added a comment -

          Folks, is there any reason behind limiting this amount of memory to 2GB ?

          Show
          milindb Milind Bhandarkar added a comment - Folks, is there any reason behind limiting this amount of memory to 2GB ?
          Hide
          vgogate@gopivotal.com Vitthal Gogate added a comment -

          I changed the code to make it long and it lowered the disk spills considerably when enough memory is given to reducers. There are some other places where Interger.MAX_VALUE is used which need be corrected.

          In MergeManagerImpl.java following code is changed to use Long value,

          • int maxInMemReduce = (int)Math.min(
          • Runtime.getRuntime().maxMemory() * maxRedPer, Integer.MAX_VALUE);

          +long maxInMemReduce = Math.min(
          + (long)(Runtime.getRuntime().maxMemory() * maxRedPer), Long.MAX_VALUE);

          I will submit the patch soon.

          Show
          vgogate@gopivotal.com Vitthal Gogate added a comment - I changed the code to make it long and it lowered the disk spills considerably when enough memory is given to reducers. There are some other places where Interger.MAX_VALUE is used which need be corrected. In MergeManagerImpl.java following code is changed to use Long value, int maxInMemReduce = (int)Math.min( Runtime.getRuntime().maxMemory() * maxRedPer, Integer.MAX_VALUE); +long maxInMemReduce = Math.min( + (long)(Runtime.getRuntime().maxMemory() * maxRedPer), Long.MAX_VALUE); I will submit the patch soon.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Vitthal Gogate, Milind Bhandarkar, are you going to submit the patch?

          Show
          jira.shegalov Gera Shegalov added a comment - Vitthal Gogate , Milind Bhandarkar , are you going to submit the patch?
          Hide
          vgogate@gopivotal.com Vitthal Gogate added a comment -

          Sorry for putting it on a back-burner, but will submit the patch soon. --Suhas

          Show
          vgogate@gopivotal.com Vitthal Gogate added a comment - Sorry for putting it on a back-burner, but will submit the patch soon. --Suhas
          Hide
          in-chief Xiaoshuang LU added a comment -

          $ find . -name "*.java" | xargs grep -Eni "Runtime.*getRuntime.*maxMemory"
          $ find . -name "*.java" | grep "mapreduce" | xargs grep -Eni -A16 -B16 "Integer.MAX_VALUE" > result

          Show
          in-chief Xiaoshuang LU added a comment - $ find . -name "*.java" | xargs grep -Eni "Runtime.*getRuntime.*maxMemory" $ find . -name "*.java" | grep "mapreduce" | xargs grep -Eni -A16 -B16 "Integer.MAX_VALUE" > result
          Hide
          jira.shegalov Gera Shegalov added a comment -

          001 for review, combining with MAPREDUCE-4883

          Show
          jira.shegalov Gera Shegalov added a comment - 001 for review, combining with MAPREDUCE-4883
          Hide
          hadoopqa Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12725123/MAPREDUCE-5649.001.patch
          against trunk revision 838b06a.

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

          +1 tests included. The patch appears to include 1 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core.

          Test results: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5400//testReport/
          Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5400//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12725123/MAPREDUCE-5649.001.patch against trunk revision 838b06a. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core. Test results: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5400//testReport/ Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5400//console This message is automatically generated.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Jason Lowe, Karthik Kambatla, please review if you get a chance.

          Show
          jira.shegalov Gera Shegalov added a comment - Jason Lowe , Karthik Kambatla , please review if you get a chance.
          Hide
          jlowe Jason Lowe added a comment -

          In general looks OK to me. Couple of nits:

          We're taking a transient computation and storing it long-term apparently only for unit testing purposes. I think we can unit test it in a way that doesn't require long-term storage. We could allow createInMemorySegments to be overridden in a derived test class so we could spy on the parameters to verify the value comes out as expected.

          This runtime exception message is not going to be very user-friendly. I realize it did not originate with this patch, but since we're touching that code it would be nice to clean it up a bit:

          +    if (maxRedPer > 1.0 || maxRedPer < 0.0) {
          +      throw new RuntimeException(MRJobConfig.REDUCE_INPUT_BUFFER_PERCENT +
          +          maxRedPer);
          +    }
          
          Show
          jlowe Jason Lowe added a comment - In general looks OK to me. Couple of nits: We're taking a transient computation and storing it long-term apparently only for unit testing purposes. I think we can unit test it in a way that doesn't require long-term storage. We could allow createInMemorySegments to be overridden in a derived test class so we could spy on the parameters to verify the value comes out as expected. This runtime exception message is not going to be very user-friendly. I realize it did not originate with this patch, but since we're touching that code it would be nice to clean it up a bit: + if (maxRedPer > 1.0 || maxRedPer < 0.0) { + throw new RuntimeException(MRJobConfig.REDUCE_INPUT_BUFFER_PERCENT + + maxRedPer); + }
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Thanks for review, Jason Lowe! 002.patch

          Show
          jira.shegalov Gera Shegalov added a comment - Thanks for review, Jason Lowe ! 002.patch
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 46s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 javac 7m 29s There were no new javac warning messages.
          +1 javadoc 9m 29s There were no new javadoc warning messages.
          +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 0m 54s There were no new checkstyle issues.
          -1 whitespace 0m 1s The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix.
          +1 install 1m 33s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 1m 15s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 1m 36s Tests passed in hadoop-mapreduce-client-core.
              38m 4s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729903/MAPREDUCE-5649.002.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / f1a152c
          whitespace https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/artifact/patchprocess/whitespace.txt
          hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/testReport/
          Java 1.7.0_55
          uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 46s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 29s There were no new javac warning messages. +1 javadoc 9m 29s There were no new javadoc warning messages. +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 54s There were no new checkstyle issues. -1 whitespace 0m 1s The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 1m 15s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 1m 36s Tests passed in hadoop-mapreduce-client-core.     38m 4s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729903/MAPREDUCE-5649.002.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / f1a152c whitespace https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/artifact/patchprocess/whitespace.txt hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/testReport/ Java 1.7.0_55 uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5487/console This message was automatically generated.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          whitespace fix

          Show
          jira.shegalov Gera Shegalov added a comment - whitespace fix
          Hide
          hadoopqa Hadoop QA added a comment -



          +1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 59s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 javac 7m 43s There were no new javac warning messages.
          +1 javadoc 9m 48s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 0m 48s There were no new checkstyle issues.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 install 1m 35s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 1m 17s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 1m 41s Tests passed in hadoop-mapreduce-client-core.
              38m 51s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729994/MAPREDUCE-5649.003.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 8f65c79
          hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/testReport/
          Java 1.7.0_55
          uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 59s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 43s There were no new javac warning messages. +1 javadoc 9m 48s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 48s There were no new checkstyle issues. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 35s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 1m 17s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 1m 41s Tests passed in hadoop-mapreduce-client-core.     38m 51s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729994/MAPREDUCE-5649.003.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 8f65c79 hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/testReport/ Java 1.7.0_55 uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5633/console This message was automatically generated.
          Hide
          hadoopqa Hadoop QA added a comment -



          +1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 35s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 javac 7m 27s There were no new javac warning messages.
          +1 javadoc 9m 35s There were no new javadoc warning messages.
          +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 0m 45s There were no new checkstyle issues.
          +1 whitespace 0m 1s The patch has no lines that end in whitespace.
          +1 install 1m 33s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 1m 16s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 1m 38s Tests passed in hadoop-mapreduce-client-core.
              37m 50s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729994/MAPREDUCE-5649.003.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 8f65c79
          hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/testReport/
          Java 1.7.0_55
          uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 35s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 27s There were no new javac warning messages. +1 javadoc 9m 35s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 45s There were no new checkstyle issues. +1 whitespace 0m 1s The patch has no lines that end in whitespace. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 1m 16s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 1m 38s Tests passed in hadoop-mapreduce-client-core.     37m 50s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729994/MAPREDUCE-5649.003.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 8f65c79 hadoop-mapreduce-client-core test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/artifact/patchprocess/testrun_hadoop-mapreduce-client-core.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/testReport/ Java 1.7.0_55 uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5635/console This message was automatically generated.
          Hide
          jlowe Jason Lowe added a comment -

          +1 lgtm. Committing this.

          Show
          jlowe Jason Lowe added a comment - +1 lgtm. Committing this.
          Hide
          jlowe Jason Lowe added a comment -

          Thanks, Gera! I committed this to trunk and branch-2.

          Show
          jlowe Jason Lowe added a comment - Thanks, Gera! I committed this to trunk and branch-2.
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #7725 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7725/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7725 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7725/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Thanks Jason Lowe for review and commit.

          Thanks Jerry Chen for your patch in MAPREDUCE-4883, I resolve it as a duplicate.

          Show
          jira.shegalov Gera Shegalov added a comment - Thanks Jason Lowe for review and commit. Thanks Jerry Chen for your patch in MAPREDUCE-4883 , I resolve it as a duplicate.
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #184 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/184/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #184 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/184/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Yarn-trunk #918 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/918/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #918 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/918/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #2116 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2116/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #2116 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2116/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #175 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/175/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #175 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/175/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #185 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/185/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #185 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/185/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #2134 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2134/)
          MAPREDUCE-5649. Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2134 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2134/ ) MAPREDUCE-5649 . Reduce cannot use more than 2G memory for the final merge. Contributed by Gera Shegalov (jlowe: rev 7dc3c1203d1ab14c09d0aaf0869a5bcdfafb0a5a) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/test/java/org/apache/hadoop/mapreduce/task/reduce/TestMergeManager.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/MergeManagerImpl.java
          Hide
          vinodkv Vinod Kumar Vavilapalli added a comment -

          Pulled this into 2.6.1. Ran compilation and TestMergeManager before the push. Patch applied cleanly.

          Show
          vinodkv Vinod Kumar Vavilapalli added a comment - Pulled this into 2.6.1. Ran compilation and TestMergeManager before the push. Patch applied cleanly.
          Hide
          vinodkv Vinod Kumar Vavilapalli added a comment -

          Just pulled this into branch-2.7 (release 2.7.2) as it already exists in 2.6.1.

          branch-2 patch applies cleanly. Ran compilation and TestMergeManager before the push.

          Show
          vinodkv Vinod Kumar Vavilapalli added a comment - Just pulled this into branch-2.7 (release 2.7.2) as it already exists in 2.6.1. branch-2 patch applies cleanly. Ran compilation and TestMergeManager before the push.

            People

            • Assignee:
              jira.shegalov Gera Shegalov
              Reporter:
              stanley_shi stanley shi
            • Votes:
              0 Vote for this issue
              Watchers:
              15 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development