You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by dd...@apache.org on 2008/03/25 08:01:29 UTC

svn commit: r640721 - in /hadoop/core/trunk: CHANGES.txt src/java/org/apache/hadoop/fs/FileSystem.java

Author: ddas
Date: Tue Mar 25 00:01:28 2008
New Revision: 640721

URL: http://svn.apache.org/viewvc?rev=640721&view=rev
Log:
HADOOP-3027. Fixes a problem to do with adding a shutdown hook in FileSystem. Contributed by Amareshwari Sriramadasu.

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/fs/FileSystem.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=640721&r1=640720&r2=640721&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Tue Mar 25 00:01:28 2008
@@ -389,6 +389,9 @@
     HADOOP-3039. Fixes a problem to do with exceptions in tasks not
     killing jobs. (Amareshwari Sriramadasu via ddas)
 
+    HADOOP-3027. Fixes a problem to do with adding a shutdown hook in
+    FileSystem.  (Amareshwari Sriramadasu via ddas)
+
 Release 0.16.1 - 2008-03-13
 
   INCOMPATIBLE CHANGES

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/fs/FileSystem.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/fs/FileSystem.java?rev=640721&r1=640720&r2=640721&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/fs/FileSystem.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/fs/FileSystem.java Tue Mar 25 00:01:28 2008
@@ -1294,11 +1294,10 @@
     synchronized FileSystem get(URI uri, Configuration conf) throws IOException{
       FileSystem fs = map.get(new Key(uri, conf));
       if (fs == null) {
+        fs = createFileSystem(uri, conf);
         if (map.isEmpty() && !clientFinalizer.isAlive()) {
           Runtime.getRuntime().addShutdownHook(clientFinalizer);
         }
-
-        fs = createFileSystem(uri, conf);
         map.put(new Key(fs), fs);
       }
       return fs;