proxy_balancer_method

[jean-frederic clere <jf...@gmail.com>]

Hi,

I would like to change the proxy_balancer_method to the following:
+++
struct proxy_balancer_method {
     const char *name;            /* name of the load balancer method*/
     proxy_worker *(*finder)(proxy_balancer *balancer,
                             request_rec *r);
     apr_status_t (*reset)(proxy_balancer *balancer, server_rec *r);
     apr_status_t (*age)(proxy_balancer *balancer, server_rec *r);
     void            *context;   /* general purpose storage */
};
+++
I want to call the reset from the child_init of mod_proxy_balancer, to 
create(or not) the workers the way the balancer logic need them.
Additionally the age has no reason to be called to during a request 
(even if we will probably call it during a request in httpd-2.2.x if we 
port that to httpd-2.2.x).

Comments?

Cheers

Jean-Frederic

Re: proxy_balancer_method

[Jim Jagielski <ji...@jaguNET.com>]

Ahh... I *did* miss it :)

+1.

PS: I should have added age and reset to the end of the struct
instead of the middle, so you can fold that in as well.

On May 12, 2009, at 8:57 AM, jean-frederic clere wrote:

> Jim Jagielski wrote:
>> I must be missing it, but I don't see the diff between the below and
>> what is in mod_proxy.h in trunk...
>
> server_rec *r instead request_rec *r for reset() and age().
>
> Cheers
>
> Jean-Frederic
>

Re: proxy_balancer_method

[jean-frederic clere <jf...@gmail.com>]

Jim Jagielski wrote:
> I must be missing it, but I don't see the diff between the below and
> what is in mod_proxy.h in trunk...

server_rec *r instead request_rec *r for reset() and age().

Cheers

Jean-Frederic

Re: proxy_balancer_method

[Jim Jagielski <ji...@jaguNET.com>]

I must be missing it, but I don't see the diff between the below and
what is in mod_proxy.h in trunk...

On May 12, 2009, at 5:47 AM, jean-frederic clere wrote:

> Hi,
>
> I would like to change the proxy_balancer_method to the following:
> +++
> struct proxy_balancer_method {
>    const char *name;            /* name of the load balancer method*/
>    proxy_worker *(*finder)(proxy_balancer *balancer,
>                            request_rec *r);
>    apr_status_t (*reset)(proxy_balancer *balancer, server_rec *r);
>    apr_status_t (*age)(proxy_balancer *balancer, server_rec *r);
>    void            *context;   /* general purpose storage */
> };
> +++
> I want to call the reset from the child_init of mod_proxy_balancer,  
> to create(or not) the workers the way the balancer logic need them.
> Additionally the age has no reason to be called to during a request  
> (even if we will probably call it during a request in httpd-2.2.x if  
> we port that to httpd-2.2.x).
>
> Comments?
>
> Cheers
>
> Jean-Frederic
>