tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r957830 - in /tomcat/trunk: java/org/apache/catalina/loader/LocalStrings.properties java/org/apache/catalina/loader/WebappClassLoader.java webapps/docs/config/context.xml
Date Sat, 26 Jun 2010 07:01:28 GMT
On 25/06/2010 11:41, Konstantin Kolinko wrote:
> 2010/6/25<markt@apache.org>:
>> Author: markt
>> Date: Fri Jun 25 07:53:24 2010
>> New Revision: 957830
>>
>> URL: http://svn.apache.org/viewvc?rev=957830&view=rev
>> Log:
>> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49414
>> Try to differentiate between request threads and threads started by applications
>>
>
>> +    /*
>> +     * Look at a threads stack trace to see if it is a request thread or not. It
>> +     * isn't perfect, but it should be good-enough for most cases.
>> +     */
>> +    private boolean isRequestThread(Thread thread) {
>> (...)
>> +        // Step through the methods in reverse order looking for a
>> +        // CoyoteAdapter.service() call. All requests will have this unless
>
> There are also
> CoyoteAdapter.event()
> CoyoteAdapter.asyncDispatch()

Thanks for the catch. I think I'll drop the check of the method name. 
Passing through CoyoteAdapter should be enough.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message