Ruediger Pluem wrote:
On 08/04/2008 03:50 PM, Mladen Turk wrote:
[EMAIL PROTECTED] wrote:
--- httpd/httpd/trunk/modules/proxy/proxy_util.c (original)
+++ httpd/httpd/trunk/modules/proxy/proxy_util.c Mon Aug 4 05:53:50
2008
@@ -1380,7 +1380,6 @@
proxy_worker *worker = (proxy_worker *)theworker;
if (worker->cp->res) {
worker->cp->pool = NULL;
- apr_reslist_destroy(worker->cp->res);
}
return APR_SUCCESS;
}
Well, you've beat me :)
The complete conn_pool_cleanup function can be omitted, and
there is no need for checking if (conn->worker->cp->pool)
in the connection_destructor.
Might be. At the current state it makes it possible to keep
trunk and 2.2.x very similar until all in apr land is sorted
out. We can cleanup the remaining parts then.
Well I was hopping to add an extra apr_reslist_maintain function
that would allow to close the expired resources out of the
require/release bounds. Right now in proxy the connection will
be release only if the particular mpm child process gets
request for a particular worker. It can take minutes or hours
on slow hit ratio for that to happen. So the ttl is more a 'guess'
rather then a firm fact.
Regards
--
^(TM)
