You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by er...@apache.org on 2013/02/08 19:06:06 UTC

git commit: GIRAPH-499: Giraph should not reserve minimum reduce slot memory 1024 since we never use it. (ereisman)

Updated Branches:
  refs/heads/trunk 8c86fa656 -> 86c2f657f


GIRAPH-499: Giraph should not reserve minimum reduce slot memory 1024 since we never use it. (ereisman)


Project: http://git-wip-us.apache.org/repos/asf/giraph/repo
Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/86c2f657
Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/86c2f657
Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/86c2f657

Branch: refs/heads/trunk
Commit: 86c2f657f5141f4d9f01934dddef8253b4277c21
Parents: 8c86fa6
Author: Eli Reisman <er...@apache.org>
Authored: Fri Feb 8 10:05:20 2013 -0800
Committer: Eli Reisman <er...@apache.org>
Committed: Fri Feb 8 10:05:20 2013 -0800

----------------------------------------------------------------------
 CHANGELOG                                          |    2 ++
 .../main/java/org/apache/giraph/job/GiraphJob.java |    2 +-
 2 files changed, 3 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/giraph/blob/86c2f657/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index bd4ef0d..3524ae3 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,8 @@
 Giraph Change Log
 
 Release 0.2.0 - unreleased
+  GIRAPH-499: Giraph should not reserve minimum reduce slot memory 1024 since we never use it (ereisman)
+
   GIRAPH-508: Increase the limit on the number of partitions (majakabiljo)
 
   GIRAPH-509: Factor out AggregatorUsage (majakabiljo)

http://git-wip-us.apache.org/repos/asf/giraph/blob/86c2f657/giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java
----------------------------------------------------------------------
diff --git a/giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java b/giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java
index 62498c6..9f711da 100644
--- a/giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java
+++ b/giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java
@@ -237,7 +237,7 @@ public class GiraphJob {
     // Capacity scheduler-specific settings.  These should be enough for
     // a reasonable Giraph job
     setIntConfIfDefault("mapred.job.map.memory.mb", 1024);
-    setIntConfIfDefault("mapred.job.reduce.memory.mb", 1024);
+    setIntConfIfDefault("mapred.job.reduce.memory.mb", 0);
 
     // Speculative execution doesn't make sense for Giraph
     giraphConfiguration.setBoolean(