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);
+ }
}