Re: proxy balancer rework

2005-08-10 Thread William A. Rowe, Jr.
At 12:07 PM 8/10/2005, Jim Jagielski wrote: >Mladen Turk wrote: >> >> Anyhow, I think that all balancer methods should be >> threated just like a protocols. If you have >> a new XXX protocol, you will need to add the XXX >> proxy protocol handler. >> >I just feel that being able to add a 'byrando

Re: proxy balancer rework

2005-08-10 Thread Jim Jagielski
Mladen Turk wrote: > > Jim Jagielski wrote: > > > > I wasn't. It was simply that we appeared to be discussing > > 2 different things. > > > > Bet I'm older and more tired than you are ;) > > > > Right. Ever visited Vukovar? Got me there :) > > Anyhow, I think that all balancer methods shoul

Re: proxy balancer rework

2005-08-10 Thread Mladen Turk
Jim Jagielski wrote: I wasn't. It was simply that we appeared to be discussing 2 different things. Bet I'm older and more tired than you are ;) Right. Ever visited Vukovar? Anyhow, I think that all balancer methods should be threated just like a protocols. If you have a new XXX protocol, yo

Re: proxy balancer rework

2005-08-10 Thread Jim Jagielski
Mladen Turk wrote: > > Jim Jagielski wrote: > > > > Did you even look at the patch? Your comments show > > a supreme lack of understanding what it does. > > > > Look Jim, I'm too old and too tired for such discussions. > If you think your patch will make the Earth a better place > to live on, y

Re: proxy balancer rework

2005-08-10 Thread Mladen Turk
Jim Jagielski wrote: Did you even look at the patch? Your comments show a supreme lack of understanding what it does. Look Jim, I'm too old and too tired for such discussions. If you think your patch will make the Earth a better place to live on, you have my +1. In future, please do not take

Re: proxy balancer rework

2005-08-10 Thread Jim Jagielski
Mladen Turk wrote: > > Jim Jagielski wrote: > >> Jim Jagielski wrote: > >> > > > > I don't know what you mean... > > Well, IMO if there is a need for a new balancer > one should write a new balancer module. > Not balancer. Method. > Right now we have both API and functionality > inside mod_pr

Re: proxy balancer rework

2005-08-10 Thread Mladen Turk
Jim Jagielski wrote: Jim Jagielski wrote: I don't know what you mean... Well, IMO if there is a need for a new balancer one should write a new balancer module. Right now we have both API and functionality inside mod_proxy that enables one to do that. If we start adding hooks to the balance

Re: proxy balancer rework

2005-08-10 Thread Jim Jagielski
On Aug 10, 2005, at 11:24 AM, Mladen Turk wrote: Jim Jagielski wrote: I've never been all that happy with the current setup in having load balance methods be so hard coded into the proxy module. Adding another method requires too many changes. Right, but mod_proxy_balancer was not meant to

Re: proxy balancer rework

2005-08-10 Thread Mladen Turk
Jim Jagielski wrote: I've never been all that happy with the current setup in having load balance methods be so hard coded into the proxy module. Adding another method requires too many changes. Right, but mod_proxy_balancer was not meant to be a generic placeholder for 'anything-you-may-ever-

proxy balancer rework

2005-08-10 Thread Jim Jagielski
I've never been all that happy with the current setup in having load balance methods be so hard coded into the proxy module. Adding another method requires too many changes. So I've refactored those sections to allow for additional lb methods to be added via simple submodules. Prelim Patch is av