https://bz.apache.org/bugzilla/show_bug.cgi?id=64338

--- Comment #3 from Ruediger Pluem <[email protected]> ---
Thanks for the patch. Some points:

1. Why not setting this environment variable in the same block as
BALANCER_WORKER_ROUTE (line 622) but only when a session route was supplied?
2. I haven't checked it we could run into issues as apr_itoa takes an int while
worker->s->status is an unsigned int. At least this could cause compiler
warnings.
3. Please add a documentation for your new variable to
docs/manual/mod/mod_proxy_balancer.xml at the section environment starting at
about line 130.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to