Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-08-02 Thread Ruediger Pluem
On 08/01/2006 01:48 PM, Jim Jagielski wrote: =?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VF_EITO?= wrote: -Urspr=FCngliche Nachricht- Von: Ruediger Pluem=20 =20 BTW: Don't we need to reset checked_standby and=20 checking_standby to zero in the outer while loop? What about this? I

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-08-01 Thread Plüm , Rüdiger , VF EITO
-Ursprüngliche Nachricht- Von: Ruediger Pluem BTW: Don't we need to reset checked_standby and checking_standby to zero in the outer while loop? What about this? I guess we don't enter the inner while loop a second time because checked_standby 0 once we have checked for

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-08-01 Thread Jim Jagielski
=?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VF_EITO?= wrote: -Urspr=FCngliche Nachricht- Von: Ruediger Pluem=20 =20 BTW: Don't we need to reset checked_standby and=20 checking_standby to zero in the outer while loop? What about this? I guess we don't enter the inner

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Jim Jagielski
Mladen Turk wrote: [EMAIL PROTECTED] wrote: Author: jim Compiles/builds clean: passes test framework as well as more normal usage tests ;) -chartimeout_set; +chartimeout_set; -characquire_set; -apr_size_t

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

2006-07-31 Thread Ruediger Pluem
On 07/31/2006 07:01 PM, [EMAIL PROTECTED] wrote: Author: jim Date: Mon Jul 31 10:01:40 2006 New Revision: 427172 URL: http://svn.apache.org/viewvc?rev=427172view=rev Log: Add in a very simple balancer set concept, which allows for members to be assigned to a particular cluster set such

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Jim Jagielski
Ruediger Pluem wrote: Shouldn't that be while (cur_lbset = max_lbset !mycandidate); (same question also for the other algorithm)? I guess otherwise we would not check for the workers with the lbset max_lbset. No, since we do the test at the end, after we've incremented. If the current

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Ruediger Pluem
On 07/31/2006 09:53 PM, Jim Jagielski wrote: Ruediger Pluem wrote: Shouldn't that be while (cur_lbset = max_lbset !mycandidate); (same question also for the other algorithm)? I guess otherwise we would not check for the workers with the lbset max_lbset. No, since we do the test at the

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Jim Jagielski
Let me double check... -- === Jim Jagielski [|] [EMAIL PROTECTED] [|] http://www.jaguNET.com/ If you can dodge a wrench, you can dodge a ball.

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Jim Jagielski
Good catch by Ruediger. Fixed. Jim Jagielski wrote: Let me double check... -- === Jim Jagielski [|] [EMAIL PROTECTED] [|] http://www.jaguNET.com/ If you can dodge a wrench, you can dodge a