svn commit: r492248 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/dfs/FSNamesystem.java

[cu...@apache.org]

Author: cutting
Date: Wed Jan  3 10:47:48 2007
New Revision: 492248

URL: http://svn.apache.org/viewvc?view=rev&rev=492248
Log:
HADOOP-745.  Fix a synchronization bug in the HDFS namenode.  Contributed by Dhruba.

Modified:
    lucene/hadoop/trunk/CHANGES.txt
    lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?view=diff&rev=492248&r1=492247&r2=492248
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Wed Jan  3 10:47:48 2007
@@ -159,6 +159,9 @@
 44. HADOOP-849.  Fix OutOfMemory exceptions in TaskTracker due to a
     file handle leak in SequenceFile.  (Devaraj Das via cutting)
 
+45. HADOOP-745.  Fix a synchronization bug in the HDFS namenode.
+    (Dhruba Borthakur via cutting)
+
 
 Release 0.9.2 - 2006-12-15
 

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java?view=diff&rev=492248&r1=492247&r2=492248
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java Wed Jan  3 10:47:48 2007
@@ -780,7 +780,7 @@
     /**
      * Change the indicated filename.
      */
-    public boolean renameTo(UTF8 src, UTF8 dst) throws IOException {
+    public synchronized boolean renameTo(UTF8 src, UTF8 dst) throws IOException {
         NameNode.stateChangeLog.debug("DIR* NameSystem.renameTo: " + src + " to " + dst );
         if( isInSafeMode() )
           throw new SafeModeException( "Cannot rename " + src, safeMode );
@@ -869,7 +869,7 @@
     /**
      * Create all the necessary directories
      */
-    public boolean mkdirs( String src ) throws IOException {
+    public synchronized boolean mkdirs( String src ) throws IOException {
         boolean    success;
         NameNode.stateChangeLog.debug("DIR* NameSystem.mkdirs: " + src );
         if( isInSafeMode() )