Author: rangadi
Date: Wed Nov 26 11:36:47 2008
New Revision: 720958
URL: http://svn.apache.org/viewvc?rev=720958&view=rev
Log:
HADOOP-4542. TestDistributedUpgrade used succeed for wrong reasons. (Raghu
Angadi)
Modified:
hadoop/core/branches/branch-0.18/CHANGES.txt
hadoop/core/branches/branch-0.18/src/test/org/apache/hadoop/dfs/TestDistributedUpgrade.java
Modified: hadoop/core/branches/branch-0.18/CHANGES.txt
URL:
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.18/CHANGES.txt?rev=720958&r1=720957&r2=720958&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.18/CHANGES.txt (original)
+++ hadoop/core/branches/branch-0.18/CHANGES.txt Wed Nov 26 11:36:47 2008
@@ -46,6 +46,9 @@
HADOOP-4614. Lazily open segments when merging map spills to avoid using
too many file descriptors. (Yuri Pradkin via cdouglas)
+ HADOOP-4542. TestDistributedUpgrade used succeed for wrong reasons.
+ (Raghu Angadi)
+
Release 0.18.2 - 2008-11-03
BUG FIXES
Modified:
hadoop/core/branches/branch-0.18/src/test/org/apache/hadoop/dfs/TestDistributedUpgrade.java
URL:
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.18/src/test/org/apache/hadoop/dfs/TestDistributedUpgrade.java?rev=720958&r1=720957&r2=720958&view=diff
==============================================================================
---
hadoop/core/branches/branch-0.18/src/test/org/apache/hadoop/dfs/TestDistributedUpgrade.java
(original)
+++
hadoop/core/branches/branch-0.18/src/test/org/apache/hadoop/dfs/TestDistributedUpgrade.java
Wed Nov 26 11:36:47 2008
@@ -51,7 +51,11 @@
*/
void startNameNodeShouldFail(StartupOption operation) {
try {
- cluster = new MiniDFSCluster(conf, 0, operation); // should fail
+ //cluster = new MiniDFSCluster(conf, 0, operation); // should fail
+ // we set manage dirs to true as NN has to start from untar'ed image
with
+ // nn dirs set to name1 and name2
+ cluster = new MiniDFSCluster(0, conf, 0, false, true,
+ operation, null); // Should fail
throw new AssertionError("NameNode should have failed to start");
} catch (Exception expected) {
expected = null;
