[jboss-cvs] JBoss Messaging SVN: r2918 - trunk/tests/src/org/jboss/test/messaging/tools.

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

Author: timfox
Date: 2007-07-19 19:23:35 -0400 (Thu, 19 Jul 2007)
New Revision: 2918

Modified:
   trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java
Log:
Fixed poison


Modified: trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java	2007-07-19 23:03:01 UTC (rev 2917)
+++ trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java	2007-07-19 23:23:35 UTC (rev 2918)
@@ -666,11 +666,6 @@
       insureStarted(serverIndex);
       Server poisoned = servers[serverIndex].getServer();
 
-      // TODO (ovidiu): this is prone to race conditions, as somebody from the client may try to
-      //       use (and create) an new server that is being poisoned, while the poisoned server is
-      //       still alive.
-
-      servers[serverIndex] = null;
       poisoned.poisonTheServer(type);
 
       return poisoned;