AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-15 Thread Plüm , Rüdiger , VIS
-Ursprüngliche Nachricht- Von: Jim Jagielski No regressions... Now that this has been passed successfully, do you see need for any further discussion / changes before I commit it or should I commit to the trunk and we continue our further changes / discussions there? Regards

Re: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-15 Thread Jim Jagielski
=?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VIS?= wrote: Von: Jim Jagielski=20 =20 No regressions... =20 Now that this has been passed successfully, do you see need for any further discussion / changes before I commit it or should I commit to the trunk and we continue our further

AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-15 Thread Plüm , Rüdiger , VIS
-Ursprüngliche Nachricht- Von: Jim Jagielski I vote to commit and use that as the continue point for more development :) Excellent. I will do so tonight German time. Currently I am away from my development env as you may have noticed from my nicely formated Outlook mails :-).

Re: AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-15 Thread Jim Jagielski
I'll do it, no prob. =?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VIS?= wrote: -Urspr=FCngliche Nachricht- Von: Jim Jagielski=20 =20 =20 I vote to commit and use that as the continue point for more development :) Excellent. I will do so tonight German time. Currently I am

AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Plüm , Rüdiger , VIS
-Ursprüngliche Nachricht- Von: Jim Jagielski On Feb 13, 2006, at 6:25 PM, Ruediger Pluem wrote: Although it's not really documented anyplace, it really is good practice for people who submit large changes to run That seems a reasonable good idea. them through the

Re: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Jim Jagielski
=?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VIS?= wrote: Currently I am away from my developing environment, but as soon as I get there (tonight German time) I will sent an updated version. Thanks in advance for running the tests. Anytime! :) I'm currently trying to trace through exactly how

AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Plüm , Rüdiger , VIS
-Ursprüngliche Nachricht- Von: Jim Jagielski I'm currently trying to trace through exactly how the code is trying to pool connections. Of course, we're only using reslist if we're a threaded MPM... Really? I thought APR_HAS_THREADS is set when the OS supports threads. I thought

Re: AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Jim Jagielski
=?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VIS?= wrote: -Urspr=FCngliche Nachricht- Von: Jim Jagielski I'm currently trying to trace through exactly how the code is=20 trying to pool connections. Of course, we're only using=20 reslist if we're a threaded MPM... Really? I

AW: AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Plüm , Rüdiger , VIS
-Ursprüngliche Nachricht- Von: Jim Jagielski Yeah, but we check to see if we're 1 thread, so in prefork, we drop to single connection workers. Which makes sense to me. Why have more than one connection per worker on a prefork processes that can only handle one request at a

Re: AW: AW: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Jim Jagielski
=?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VIS?= wrote: -Urspr=FCngliche Nachricht- Von: Jim Jagielski=20 =20 =20 Yeah, but we check to see if we're 1 thread, so in prefork, we drop to single connection workers. Which makes sense to me. To me too. What it's doing is

Re: AW: [Patch] Keep Alive not workwing with mod_proxy (PR38602)

2006-02-14 Thread Brian Akins
Jim Jagielski wrote: I'm currently trying to trace through exactly how the code is trying to pool connections. If someone produces a good patch, I have some traffic I can throw at it :) -- Brian Akins Lead Systems Engineer CNN Internet Technologies