Repository: crunch
Updated Branches:
  refs/heads/apache-crunch-0.8 a98c0e5d5 -> 74f512ccb


CRUNCH-463: Don't copy the Configuration object during CrunchInputSplit 
initialization


Project: http://git-wip-us.apache.org/repos/asf/crunch/repo
Commit: http://git-wip-us.apache.org/repos/asf/crunch/commit/74f512cc
Tree: http://git-wip-us.apache.org/repos/asf/crunch/tree/74f512cc
Diff: http://git-wip-us.apache.org/repos/asf/crunch/diff/74f512cc

Branch: refs/heads/apache-crunch-0.8
Commit: 74f512ccb9bc1380e95b004836c3bfd418ee519e
Parents: a98c0e5
Author: Josh Wills <[email protected]>
Authored: Sat Aug 16 10:13:21 2014 -0700
Committer: Josh Wills <[email protected]>
Committed: Tue Aug 19 10:42:40 2014 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java | 2 +-
 .../main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/crunch/blob/74f512cc/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java
----------------------------------------------------------------------
diff --git 
a/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java
 
b/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java
index 8c173cc..11e1038 100644
--- 
a/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java
+++ 
b/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputFormat.java
@@ -64,7 +64,7 @@ public class CrunchInputFormat<K, V> extends InputFormat<K, 
V> {
         // and Mapper types by wrapping in a TaggedInputSplit.
         List<InputSplit> pathSplits = format.getSplits(jobCopy);
         for (InputSplit pathSplit : pathSplits) {
-          splits.add(new CrunchInputSplit(pathSplit, inputBundle, nodeIndex, 
conf));
+          splits.add(new CrunchInputSplit(pathSplit, inputBundle, nodeIndex, 
jobCopy.getConfiguration()));
         }
       }
     }

http://git-wip-us.apache.org/repos/asf/crunch/blob/74f512cc/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java
----------------------------------------------------------------------
diff --git 
a/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java 
b/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java
index ec255d9..ee54483 100644
--- 
a/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java
+++ 
b/crunch-core/src/main/java/org/apache/crunch/impl/mr/run/CrunchInputSplit.java
@@ -55,7 +55,7 @@ class CrunchInputSplit extends InputSplit implements 
Writable, Configurable, Sup
     this.inputSplit = inputSplit;
     this.bundle = bundle;
     this.nodeIndex = nodeIndex;
-    this.conf = new Configuration(conf);
+    this.conf = conf;
   }
 
   @Override

Reply via email to