You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by ta...@apache.org on 2013/05/24 16:27:08 UTC

svn commit: r1486074 - /activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java

Author: tabish
Date: Fri May 24 14:27:08 2013
New Revision: 1486074

URL: http://svn.apache.org/r1486074
Log:
fixes for: https://issues.apache.org/jira/browse/AMQ-4555

Only create update thread on demand. 

Modified:
    activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java

Modified: activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java?rev=1486074&r1=1486073&r2=1486074&view=diff
==============================================================================
--- activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java (original)
+++ activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/security/SimpleCachedLDAPAuthorizationMap.java Fri May 24 14:27:08 2013
@@ -106,9 +106,9 @@ public class SimpleCachedLDAPAuthorizati
         new ConcurrentHashMap<ActiveMQDestination, AuthorizationEntry>();
 
     public SimpleCachedLDAPAuthorizationMap() {
-        // Allow for only a couple outstanding update requests, they can be slow so we
+        // Allow for only a couple outstanding update request, they can be slow so we
         // don't want a bunch to pile up for no reason.
-        updaterService = new ThreadPoolExecutor(1, 1, 0, TimeUnit.SECONDS,
+        updaterService = new ThreadPoolExecutor(0, 1, 60, TimeUnit.SECONDS,
             new LinkedBlockingQueue<Runnable>(2),
             new ThreadFactory() {