You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2014/01/14 20:19:44 UTC

svn commit: r1558174 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java

Author: tedyu
Date: Tue Jan 14 19:19:43 2014
New Revision: 1558174

URL: http://svn.apache.org/r1558174
Log:
HBASE-10294 ServerManager#onlineServers synchronization


Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java?rev=1558174&r1=1558173&r2=1558174&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java Tue Jan 14 19:19:43 2014
@@ -421,7 +421,9 @@ public class ServerManager {
    */
   public Map<ServerName, ServerLoad> getOnlineServers() {
     // Presumption is that iterating the returned Map is OK.
-    return Collections.unmodifiableMap(this.onlineServers);
+    synchronized (this.onlineServers) {
+      return Collections.unmodifiableMap(this.onlineServers);
+    }
   }