Author: dhruba
Date: Wed Jun 18 12:19:57 2008
New Revision: 669258

URL: http://svn.apache.org/viewvc?rev=669258&view=rev
Log:
HADOOP-3520.  TestDFSUpgradeFromImage triggers a race condition in the
Upgrade Manager. Fixed. (dhruba)



Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/hdfs/org/apache/hadoop/dfs/UpgradeManagerDatanode.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=669258&r1=669257&r2=669258&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Wed Jun 18 12:19:57 2008
@@ -622,6 +622,9 @@
     HADOOP-3539.  Fix the tool path in the bin/hadoop script under
     cygwin. (Tsz Wo (Nicholas), Sze via omalley)
 
+    HADOOP-3520.  TestDFSUpgradeFromImage triggers a race condition in the
+    Upgrade Manager. Fixed. (dhruba)
+
 Release 0.17.1 - Unreleased
 
   INCOMPATIBLE CHANGES

Modified: 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/dfs/UpgradeManagerDatanode.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/hdfs/org/apache/hadoop/dfs/UpgradeManagerDatanode.java?rev=669258&r1=669257&r2=669258&view=diff
==============================================================================
--- 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/dfs/UpgradeManagerDatanode.java 
(original)
+++ 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/dfs/UpgradeManagerDatanode.java 
Wed Jun 18 12:19:57 2008
@@ -18,7 +18,6 @@
 package org.apache.hadoop.dfs;
 
 import java.io.IOException;
-import java.net.SocketTimeoutException;
 
 import org.apache.hadoop.util.Daemon;
 
@@ -44,7 +43,7 @@
     return FSConstants.NodeType.DATA_NODE;
   }
 
-  void initializeUpgrade(NamespaceInfo nsInfo) throws IOException {
+  synchronized void initializeUpgrade(NamespaceInfo nsInfo) throws IOException 
{
     if( ! super.initializeUpgrade())
       return; // distr upgrade is not needed
     DataNode.LOG.info("\n   Distributed upgrade for DataNode " 


Reply via email to