[jboss-cvs] JBoss Messaging SVN: r4171 - in trunk: src/main/org/jboss/messaging/core/client/impl and 1 other directory.

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

Author: timfox
Date: 2008-05-12 06:03:42 -0400 (Mon, 12 May 2008)
New Revision: 4171

Modified:
   trunk/build-messaging.xml
   trunk/src/main/org/jboss/messaging/core/client/impl/ClientConsumerImpl.java
Log:
Set direct always to true


Modified: trunk/build-messaging.xml
===================================================================
--- trunk/build-messaging.xml	2008-05-09 16:50:24 UTC (rev 4170)
+++ trunk/build-messaging.xml	2008-05-12 10:03:42 UTC (rev 4171)
@@ -587,6 +587,7 @@
    <target name="unit-tests" depends="jar, compile-unit-tests">
       <echo message=""/>
       <echo message="Running unit tests, fork=${junit.fork}, junit.batchtest.fork=${junit.batchtest.fork}"/>
+      <echo message="classpath is:${toString:unit.test.execution.classpath}"/>
       <echo message=""/>
       <mkdir dir="${test.output.dir}"/>
       <mkdir dir="${test.reports.dir}"/>

Modified: trunk/src/main/org/jboss/messaging/core/client/impl/ClientConsumerImpl.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/client/impl/ClientConsumerImpl.java	2008-05-09 16:50:24 UTC (rev 4170)
+++ trunk/src/main/org/jboss/messaging/core/client/impl/ClientConsumerImpl.java	2008-05-12 10:03:42 UTC (rev 4171)
@@ -114,7 +114,9 @@
       
       this.remotingConnection = remotingConnection;
       
-      this.direct = direct;
+      //This is always true since the MinaHandler will use an executor based on session id
+      //TODO can remove this
+      this.direct = true;
       
       this.tokenBatchSize = tokenBatchSize;
    }