Re: [PATCH] Do not reuse persistent connections for PUTs

2012-09-01 Thread Alex Rousskov
On 09/01/2012 07:11 AM, Henrik Nordström wrote: > fre 2012-08-31 klockan 21:44 +0300 skrev Tsantilas Christos: > >> So looks that a good solution should be (similar to) the solution >> proposed by Henrik... > > 100 Continue aviods the race entirely on requests with bodies, leaving > only bodyless

Re: [PATCH] Do not reuse persistent connections for PUTs

2012-09-01 Thread Henrik Nordström
fre 2012-08-31 klockan 21:44 +0300 skrev Tsantilas Christos: > So looks that a good solution should be (similar to) the solution > proposed by Henrik... 100 Continue aviods the race entirely on requests with bodies, leaving only bodyless requests in the "we may not retry this on failure but we ne

Re: [PATCH] Do not reuse persistent connections for PUTs

2012-09-01 Thread Henrik Nordström
fre 2012-08-31 klockan 10:58 +0300 skrev Tsantilas Christos: > 1) To decide if it can reuse a healthy persistent connection. Inside > FwdState::connectStart, we are getting a persistant connection and even > if it is healthy, if we want to send eg a PUT request we are closing the > persistent con

Re: ICAP connections under heavy loads

2012-09-01 Thread Amos Jeffries
On 1/09/2012 5:05 a.m., Alex Rousskov wrote: On 08/31/2012 09:01 AM, Alexander Komyagin wrote: Alex, I figured it out, finally! The bug was in comm_connect_addr() function (I suppose it is kernel-dependent though). Consider following call trace: 1) Xaction starts ConnOpener in order to create n