[jboss-cvs] JBoss Messaging SVN: r5068 - branches/Branch_1_4/src/main/org/jboss/jms/client/remoting.

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

Author: clebert.suconic at jboss.com
Date: 2008-10-03 15:07:12 -0400 (Fri, 03 Oct 2008)
New Revision: 5068

Modified:
   branches/Branch_1_4/src/main/org/jboss/jms/client/remoting/JMSRemotingConnection.java
Log:
https://jira.jboss.org/jira/browse/JBMESSAGING-1132 - This warning don't make any sense on HTTP

Modified: branches/Branch_1_4/src/main/org/jboss/jms/client/remoting/JMSRemotingConnection.java
===================================================================
--- branches/Branch_1_4/src/main/org/jboss/jms/client/remoting/JMSRemotingConnection.java	2008-10-03 18:30:02 UTC (rev 5067)
+++ branches/Branch_1_4/src/main/org/jboss/jms/client/remoting/JMSRemotingConnection.java	2008-10-03 19:07:12 UTC (rev 5068)
@@ -313,16 +313,19 @@
                                             " use " + JBM_MAX_POOL_SIZE_KEY +" instead");
       }         
       
-      String val = (String)serverLocator.getParameters().get(JBM_MAX_POOL_SIZE_KEY);
-      
-      if (val == null)
+      if (!serverLocator.getProtocol().equals("http") & !serverLocator.getProtocol().equals("https"))
       {
-         log.warn(JBM_MAX_POOL_SIZE_KEY + " not specified - defaulting to 200");
+         String val = (String)serverLocator.getParameters().get(JBM_MAX_POOL_SIZE_KEY);
          
-         val = "200";
+         if (val == null)
+         {
+            log.warn(JBM_MAX_POOL_SIZE_KEY + " not specified - defaulting to 200");
+            
+            val = "200";
+         }
+         
+         config.put(MicroSocketClientInvoker.MAX_POOL_SIZE_FLAG, val);
       }
-      
-      config.put(MicroSocketClientInvoker.MAX_POOL_SIZE_FLAG, val);
 
       client = new Client(serverLocator, config);