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

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

Author: ovidiu.feodorov at jboss.com
Date: 2007-01-25 20:02:40 -0500 (Thu, 25 Jan 2007)
New Revision: 2062

Modified:
   trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java
Log:
adding a TODO note

Modified: trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java	2007-01-25 21:52:12 UTC (rev 2061)
+++ trunk/tests/src/org/jboss/test/messaging/tools/ServerManagement.java	2007-01-26 01:02:40 UTC (rev 2062)
@@ -641,6 +641,9 @@
    {
       insureStarted(serverIndex);
       servers[serverIndex].getServer().poisonTheServer(type);
+      // 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;
    }