2009/10/28  <ma...@apache.org>:
> Author: markt
> Date: Wed Oct 28 15:30:49 2009
> New Revision: 830589
>
>
> +        if (endpoint.getCurrentThreadsBusy() >0 &&
> +                endpoint.getMaxThreads() >0) {
> +            threadRatio = (endpoint.getCurrentThreadsBusy() * 100)
> +                    / endpoint.getMaxThreads();
> +        }

I think that endpoint.getCurrentThreadsBusy() call is not cheap (and
can provide different results between calls). It would be better to
cache its result in a local variable.

Best regards,
Konstantin Kolinko

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

Reply via email to