Re: [PATCH] : Count retries and redispatches also for servers + extend logs + %d->%u cleanup

On Tue, 25 Dec 2007, Krzysztof Oledzki wrote:

> From c71a482f5c303392d06c228d868a4898efde07df Mon Sep 17 00:00:00 2001
> From: Krzysztof Piotr Oledzki <ole#ans.pl>
> Date: Tue, 25 Dec 2007 04:01:58 +0100
> Subject: [MINOR]: Count retries and redispatches also for servers + extend logs + %d->%u cleanup
>
> This patch extends a little previously added functionality to also
> count retries and redispatches for servers. Now it is possible to know
> which server causes redispatches as it is not always the same that takes
> most retries.
>
> I also extended logs to show how many retries were done and if redispatching
> was necessary ('+'). Currently I'm using "s->conn_retries>0" logic to match
> redispatched connections but in the future it may be required to add for example
> additional SN_REDISP flag.

Blah, this is wrong. :( Even without redispatching s->conn_retries falls below 0 when last retry fails. So, we need this SN_REDISP anyway. Please ignore c71a482f5c303392d06c228d868a4898efde07df and 227206d2f11feb3b52909f86917f0a560d343871 for now.

I'll send updated patch.

Best regards,

                                 Krzysztof Olêdzki Received on 2007/12/25 19:54