You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rp...@apache.org on 2006/04/14 13:30:14 UTC

svn commit: r394065 - /httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

Author: rpluem
Date: Fri Apr 14 04:30:04 2006
New Revision: 394065

URL: http://svn.apache.org/viewcvs?rev=394065&view=rev
Log:
* After r377780 the condition in the if clause never becomes true, because
  ap_proxy_initialize_worker_share is always left with PROXY_WORKER_INITIALIZED
  set in workers->s->status. So remove this code to save cycles and avoid
  confusion.

Modified:
    httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c?rev=394065&r1=394064&r2=394065&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c Fri Apr 14 04:30:04 2006
@@ -88,16 +88,6 @@
 
     for (i = 0; i < balancer->workers->nelts; i++) {
         ap_proxy_initialize_worker_share(conf, workers, s);
-        if (!(workers->s->status & PROXY_WORKER_INITIALIZED)) {
-            workers->s->status |= (workers->status | PROXY_WORKER_INITIALIZED);
-            ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s,
-                         "proxy: BALANCER: initialized balancer member %d for "
-                         "balancer %s in child %" APR_PID_T_FMT " for (%s) "
-                         "min=%d max=%d smax=%d",
-                          workers->id, balancer->name, getpid(),
-                          workers->hostname, workers->min, workers->hmax,
-                          workers->smax);
-        }
         ++workers;
     }