Add volatile to methodNameMap and chainMap (#34)

Modify MethodUtil#methodNameMap and CtSph#chainMap: add `volatile` to keep in line with others
alibaba · Aug 5, 2018 · 8075232 · 8075232
1 parent b2a3ef2
commit 8075232
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/sentinel-core/src/main/java/com/alibaba/csp/sentinel/CtSph.java b/sentinel-core/src/main/java/com/alibaba/csp/sentinel/CtSph.java
@@ -48,7 +48,7 @@ public class CtSph implements Sph {
      * Same resource({@link ResourceWrapper#equals(Object)}) will share the same
      * {@link ProcessorSlotChain}, no matter in which {@link Context}.
      */
-    private static Map<ResourceWrapper, ProcessorSlotChain> chainMap
+    private static volatile Map<ResourceWrapper, ProcessorSlotChain> chainMap
         = new HashMap<ResourceWrapper, ProcessorSlotChain>();
 
     private static final Object LOCK = new Object();

diff --git a/sentinel-core/src/main/java/com/alibaba/csp/sentinel/util/MethodUtil.java b/sentinel-core/src/main/java/com/alibaba/csp/sentinel/util/MethodUtil.java
@@ -29,7 +29,7 @@
  */
 public final class MethodUtil {
 
-    private static Map<Method, String> methodNameMap = new HashMap<Method, String>();
+    private static volatile Map<Method, String> methodNameMap = new HashMap<Method, String>();
 
     private static final Object LOCK = new Object();