Re: [PATCH] Add DNS lookup step to ICAP connect

On 07/01/2011 12:07 PM, Tsantilas Christos wrote:
> The above can be modified as follows:
> - Always increase the "theBusyConns" counter inside
> "ServiceRep::getConnection" method (even if we did not return a connection)
> - (1)Define a new method "ServiceRep::noteConnectionFailed" which will
> decrease the "theBusyConns" counter and will be called from Xaction when
> the connection failed OR (2) move the "open-connection" logic in
> ServiceRep, where the theBusyConns will be decreased on connection
> failures. (The second maybe is not easy...)
> - Decrease the "theBusyConns" counter inside ServiceRep::putConnection

The 4th version of the patch, which implements the above .
It still needs some testing, I am posting it here in the case someone
wants to test it.