[jboss-remoting-commits] JBoss Remoting SVN: r6373 - remoting2/branches/2.2/src/main/org/jboss/remoting.

[jboss-remoting-commits at lists.jboss.org]

Author: ron.sigal at jboss.com
Date: 2011-05-25 11:59:36 -0400 (Wed, 25 May 2011)
New Revision: 6373

Modified:
   remoting2/branches/2.2/src/main/org/jboss/remoting/ServerInvoker.java
Log:
JBREM-1292: Made callbackHandlers a ConcurrentHashMap.

Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/ServerInvoker.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/ServerInvoker.java	2011-05-16 20:45:52 UTC (rev 6372)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/ServerInvoker.java	2011-05-25 15:59:36 UTC (rev 6373)
@@ -257,7 +257,7 @@
    // to avoid lookup in this common case - TLF
    protected volatile CallbackContainer singleCallbackContainer;
       
-   protected Map callbackHandlers = new HashMap();
+   protected Map callbackHandlers = new ConcurrentHashMap();
    protected Map clientCallbackListener = new HashMap();
    protected boolean started = false;
    protected ConnectionNotifier connectionNotifier = new ConnectionNotifier();
@@ -1973,7 +1973,7 @@
    {
       ServerInvokerCallbackHandler callbackHandler = null;
       String id = ServerInvokerCallbackHandler.getId(invocation);
-      synchronized(callbackHandlers)
+//      synchronized(callbackHandlers)
       {
          callbackHandler = (ServerInvokerCallbackHandler)callbackHandlers.get(id);
 
@@ -1995,7 +1995,7 @@
       String id = ServerInvokerCallbackHandler.getId(invocation);
       ServerInvokerCallbackHandler callbackHandler = null;
 
-      synchronized(callbackHandlers)
+//      synchronized(callbackHandlers)
       {
          callbackHandler = (ServerInvokerCallbackHandler) callbackHandlers.remove(id);
       }