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

Posted to commits@jmeter.apache.org by se...@apache.org on 2016/04/29 15:36:46 UTC

svn commit: r1741627 - /jmeter/trunk/src/core/org/apache/jmeter/JMeter.java

Author: sebb
Date: Fri Apr 29 13:36:45 2016
New Revision: 1741627

URL: http://svn.apache.org/viewvc?rev=1741627&view=rev
Log:
Document

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/JMeter.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/JMeter.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/JMeter.java?rev=1741627&r1=1741626&r2=1741627&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/JMeter.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/JMeter.java Fri Apr 29 13:36:45 2016
@@ -1067,12 +1067,13 @@ public class JMeter implements JMeterPlu
         }
 
         @Override
+        // N.B. this is called by a daemon RMI thread from the remote host
         public void testEnded(String host) {
             long now=System.currentTimeMillis();
             log.info("Finished remote host: " + host + " ("+now+")");
             if (started.decrementAndGet() <= 0) {
                 Thread stopSoon = new Thread(this);
-                stopSoon.setDaemon(false);
+                stopSoon.setDaemon(false); // the calling thread is a daemon; this thread must not be
                 stopSoon.start();
             }
         }