Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Jim Jagielski
This is uglier than creating a subpool... :) On Oct 17, 2013, at 12:10 PM, Yann Ylavic ylavic@gmail.com wrote: Maybe using the following macros could help to log worker's (full) name when no pool is available for ap_proxy_worker_name(). Regards, Index: modules/proxy/mod_proxy.h

Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Yann Ylavic
On Thu, Oct 17, 2013 at 6:21 PM, Jim Jagielski j...@jagunet.com wrote: This is uglier than creating a subpool... :) Possibly ;) but with the pool being destroyed then... Regards.

Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Yann Ylavic
On Thu, Oct 17, 2013 at 6:25 PM, Yann Ylavic ylavic@gmail.com wrote: On Thu, Oct 17, 2013 at 6:21 PM, Jim Jagielski j...@jagunet.com wrote: This is uglier than creating a subpool... :) Possibly ;) but with the pool being destroyed then... Something like : PROXY_DECLARE(char *)

Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Jim Jagielski
That's why I chose what I did... It's only needed once and the pool is around. But yeah, it may be better. On Oct 17, 2013, at 12:25 PM, Yann Ylavic ylavic@gmail.com wrote: On Thu, Oct 17, 2013 at 6:21 PM, Jim Jagielski j...@jagunet.com wrote: This is uglier than creating a subpool... :)

Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Yann Ylavic
On Thu, Oct 17, 2013 at 4:10 PM, j...@apache.org wrote: Author: jim Date: Thu Oct 17 14:10:43 2013 New Revision: 1533087 @@ -2087,12 +2089,9 @@ PROXY_DECLARE(int) ap_proxy_acquire_conn (*conn)-close = 0; (*conn)-inreslist = 0; -if (worker-s-uds) { +if

Re: svn commit: r1533087 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.h mod_proxy_balancer.c proxy_util.c

2013-10-17 Thread Jim Jagielski
For sure the (*conn)-pool is recycled more often... On Oct 17, 2013, at 3:52 PM, Yann Ylavic ylavic@gmail.com wrote: On Thu, Oct 17, 2013 at 4:10 PM, j...@apache.org wrote: Author: jim Date: Thu Oct 17 14:10:43 2013 New Revision: 1533087 @@ -2087,12 +2089,9 @@ PROXY_DECLARE(int)