[PATCH] (mod_stats) Add printing of response time

["Brett E." <br...@fastclick.com>]

This patch basically computes the average response time and adds "X 
ms/request response time" where X is the response time in milliseconds 
to the mod_status display

It works by taking the average of response times of threads who 
processed a request within the last RESPTIME_WINDOW_SECS seconds. I set 
this to 5 seconds.

I also added in the functionality to print the last response time (Req 
column), previously this was hardcoded to 0 since we didn't bother to 
fill out start_time and stop_time.

Thoughts?  This is my first patch submission so please, be easy. :) 
Maybe we weren't filling out start_time/stop_time due to excessive CPU 
caused by getting the time?

Re: [PATCH] (mod_stats) Add printing of response time

["Brett E." <br...@fastclick.com>]

Any feeling as to whether or not this can be incorporated into Apache?

Brett E. wrote:
> This patch basically computes the average response time and adds "X 
> ms/request response time" where X is the response time in milliseconds 
> to the mod_status display
> 
> It works by taking the average of response times of threads who 
> processed a request within the last RESPTIME_WINDOW_SECS seconds. I set 
> this to 5 seconds.
> 
> I also added in the functionality to print the last response time (Req 
> column), previously this was hardcoded to 0 since we didn't bother to 
> fill out start_time and stop_time.
> 
> Thoughts?  This is my first patch submission so please, be easy. :) 
> Maybe we weren't filling out start_time/stop_time due to excessive CPU 
> caused by getting the time?