You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-commits@hadoop.apache.org by om...@apache.org on 2011/03/08 07:00:57 UTC
svn commit: r1079256 - in
/hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop:
mapred/TestLazyOutput.java mapreduce/TestMapReduceLazyOutput.java
Author: omalley
Date: Tue Mar 8 06:00:57 2011
New Revision: 1079256
URL: http://svn.apache.org/viewvc?rev=1079256&view=rev
Log:
commit b43e691705e020927db4f7938d680bef30e8ea23
Author: Greg Roelofs <ro...@yahoo-inc.com>
Date: Wed Jan 26 19:48:33 2011 -0800
Coding-style fix (correct one this time) to both LazyOutput unit tests, per
Dick's final review comment.
Modified:
hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestLazyOutput.java
hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapreduce/TestMapReduceLazyOutput.java
Modified: hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestLazyOutput.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestLazyOutput.java?rev=1079256&r1=1079255&r2=1079256&view=diff
==============================================================================
--- hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestLazyOutput.java (original)
+++ hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapred/TestLazyOutput.java Tue Mar 8 06:00:57 2011
@@ -196,8 +196,12 @@ public class TestLazyOutput extends Test
numMappers, fileList.length);
} finally {
- if (dfs != null) { dfs.shutdown(); }
- if (mr != null) { mr.shutdown(); }
+ if (dfs != null) {
+ dfs.shutdown();
+ }
+ if (mr != null) {
+ mr.shutdown();
+ }
}
}
}
Modified: hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapreduce/TestMapReduceLazyOutput.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapreduce/TestMapReduceLazyOutput.java?rev=1079256&r1=1079255&r2=1079256&view=diff
==============================================================================
--- hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapreduce/TestMapReduceLazyOutput.java (original)
+++ hadoop/mapreduce/branches/yahoo-merge/src/test/mapred/org/apache/hadoop/mapreduce/TestMapReduceLazyOutput.java Tue Mar 8 06:00:57 2011
@@ -186,8 +186,12 @@ public class TestMapReduceLazyOutput ext
numMappers, fileList.length);
} finally {
- if (dfs != null) { dfs.shutdown(); }
- if (mr != null) { mr.shutdown(); }
+ if (dfs != null) {
+ dfs.shutdown();
+ }
+ if (mr != null) {
+ mr.shutdown();
+ }
}
}