You are viewing a plain text version of this content. The canonical link for it is here.

Posted to hdfs-commits@hadoop.apache.org by to...@apache.org on 2011/07/19 20:39:47 UTC

svn commit: r1148480 - /hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java

Author: todd
Date: Tue Jul 19 18:39:46 2011
New Revision: 1148480

URL: http://svn.apache.org/viewvc?rev=1148480&view=rev
Log:
Fix TestEditLog.testSimpleEditLog to not ignore errors in cluster shutdown

(in response to review by Jitendra)

Modified:
    hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java

Modified: hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java?rev=1148480&r1=1148479&r2=1148480&view=diff
==============================================================================
--- hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java (original)
+++ hadoop/common/branches/HDFS-1073/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestEditLog.java Tue Jul 19 18:39:46 2011
@@ -220,12 +220,8 @@ public class TestEditLog extends TestCas
       
       editLog.close();
     } finally {
-      try {
-        if(fileSys != null) fileSys.close();
-        if(cluster != null) cluster.shutdown();
-      } catch (Throwable t) {
-        LOG.error("Couldn't shut down", t);
-      }
+      if(fileSys != null) fileSys.close();
+      if(cluster != null) cluster.shutdown();
     }
   }