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

Posted to commits@hbase.apache.org by st...@apache.org on 2011/06/28 06:13:19 UTC

svn commit: r1140426 - /hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java

Author: stack
Date: Tue Jun 28 04:13:19 2011
New Revision: 1140426

URL: http://svn.apache.org/viewvc?rev=1140426&view=rev
Log:
HBASE-3946 broke TestMasterFailover -- remove redundant test for null storedInfo (spotted by Gao Jinchao)

Modified:
    hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java

Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java?rev=1140426&r1=1140425&r2=1140426&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java (original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java Tue Jun 28 04:13:19 2011
@@ -280,9 +280,7 @@ public class ServerManager {
           "message processing because not yet ready but possible we won't be " +
           "ready next on next report");
 
-      if (storedInfo == null) {
-        storedInfo = this.onlineServers.get(info.getServerName());
-      }
+      storedInfo = this.onlineServers.get(info.getServerName());
     }
 
     // Check startcodes