Repository: storm
Updated Branches:
  refs/heads/master 66ff5fd94 -> 8e90ffc2b


STORM-2750: fix double_checked locking

update HBaseSecurityUtil singleton to fix double_checked,  
http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html


Project: http://git-wip-us.apache.org/repos/asf/storm/repo
Commit: http://git-wip-us.apache.org/repos/asf/storm/commit/c78eb0d5
Tree: http://git-wip-us.apache.org/repos/asf/storm/tree/c78eb0d5
Diff: http://git-wip-us.apache.org/repos/asf/storm/diff/c78eb0d5

Branch: refs/heads/master
Commit: c78eb0d522e97cc00136b46e1188ba08a5ca9d9c
Parents: 559053d
Author: xiaoyong03 <[email protected]>
Authored: Wed Sep 20 16:44:24 2017 +0800
Committer: xiaoyong03 <[email protected]>
Committed: Fri Sep 22 11:55:37 2017 +0800

----------------------------------------------------------------------
 .../java/org/apache/storm/hbase/security/HBaseSecurityUtil.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/storm/blob/c78eb0d5/external/storm-autocreds/src/main/java/org/apache/storm/hbase/security/HBaseSecurityUtil.java
----------------------------------------------------------------------
diff --git 
a/external/storm-autocreds/src/main/java/org/apache/storm/hbase/security/HBaseSecurityUtil.java
 
b/external/storm-autocreds/src/main/java/org/apache/storm/hbase/security/HBaseSecurityUtil.java
index 65dcd6f..1afdf63 100644
--- 
a/external/storm-autocreds/src/main/java/org/apache/storm/hbase/security/HBaseSecurityUtil.java
+++ 
b/external/storm-autocreds/src/main/java/org/apache/storm/hbase/security/HBaseSecurityUtil.java
@@ -45,7 +45,7 @@ public class HBaseSecurityUtil {
     public static final String HBASE_KEYTAB_FILE_KEY = "hbase.keytab.file";
     public static final String HBASE_PRINCIPAL_KEY = 
"hbase.kerberos.principal";
 
-    private static UserProvider legacyProvider = null;
+    private static volatile UserProvider legacyProvider = null;
 
     private HBaseSecurityUtil() {
     }

Reply via email to