<div dir="ltr">Hi,<div><br></div><div>I guess the check of the list tail for expired ones could be moved to the BackendClose()/BackendRecycle() functions which are called after the fetch has finished and as such shouldn't create any delays for anything. Though I wouldn't expect this to have any significant impact on fetch times, it would probably be just one connection being closed at a time and compared to the fetch operation the time spent is negligible.</div>
<div><br></div><div>Having a nanny thread for this part is in my opinion too much complexity for little gain. Having a single worker servicing all of the backends would create some interesting locking complexity and resource freeing headaches. Having one per backend would be easy to implement, but would add too much resource usage for little benefit.</div>
<div><br></div><div>Nils: A review would be much appreciated.</div><div><br></div><div>Martin</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 27 August 2014 20:35, Nils Goroll <span dir="ltr"><<a href="mailto:slink@schokola.de" target="_blank">slink@schokola.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Martin,<br>
<br>
it's cool you are addressing this issue, thanks. I have not tested your patch<br>
(yet) but at first sight it looks good to me. If no-one else did a real review<br>
and phk thinks it is necessary, I could probably do it on Friday.<br>
<br>
While your patch appears to be a real improvement which I'd like to see<br>
integrated, I still think we should have some backend connection nanny thread<br>
doing this work preventatively and out of context of the backend thread<br>
desperately in need of a backend connection.<br>
<br>
Cheers, Nils<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><table border="0" cellpadding="0" cellspacing="0" style="font-size:12px;line-height:1.5em;font-family:'Helvetica Neue',Arial,sans-serif;color:rgb(102,102,102);width:550px;border-top-width:1px;border-top-style:solid;border-top-color:rgb(238,238,238);border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(238,238,238);margin-top:20px;padding-top:5px;padding-bottom:5px">
<tbody><tr><td width="100"><a href="http://varnish-software.com" target="_blank"><img src="http://www.varnish-software.com/static/media/logo-email.png"></a><span></span><span></span></td><td><strong style="font-size:14px;color:rgb(34,34,34)">Martin Blix Grydeland</strong><br>
Senior Developer | Varnish Software AS<br>Cell: +47 21 98 92 60<br><span style="font-weight:bold">We Make Websites Fly!</span></td></tr></tbody></table></div>
</div>