svn commit: r554587 - in /tomcat/tc6.0.x/trunk: java/org/apache/tomcat/util/threads/ThreadPool.java webapps/docs/changelog.xml

[rjung]

Author: rjung
Date: Mon Jul  9 02:48:48 2007
New Revision: 554587

URL: http://svn.apache.org/viewvc?view=rev&rev=554587
Log:
Separate sequence increment from getter in ThreadPool to avoid
misleading increments during monitoring via JMX.

Modified:
    tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/threads/ThreadPool.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/threads/ThreadPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/threads/ThreadPool.java?view=diff&rev=554587&r1=554586&r2=554587
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/threads/ThreadPool.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/threads/ThreadPool.java 
Mon Jul  9 02:48:48 2007
@@ -260,6 +260,10 @@
     }
 
     public int getSequence() {
+        return sequence;
+    }
+
+    public int incSequence() {
         return sequence++;
     }
 
@@ -638,7 +642,7 @@
             this.p = p;
             t = new ThreadWithAttributes(p, this);
             t.setDaemon(true);
-            t.setName(p.getName() + "-Processor" + p.getSequence());
+            t.setName(p.getName() + "-Processor" + p.incSequence());
             t.setPriority(p.getThreadPriority());
             p.addThread( t, this );
            noThData=true;

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?view=diff&rev=554587&r1=554586&r2=554587
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Mon Jul  9 02:48:48 2007
@@ -98,6 +98,10 @@
   <subsection name="Coyote">
     <changelog>
       <fix>
+        Separate sequence increment from getter in ThreadPool to avoid
+        misleading increments during monitoring via JMX. (rjung)
+      </fix>
+      <fix>
         Add back missing socketBuffer attribute in the java.io HTTP connector 
(remm)
       </fix>
     </changelog>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]