<div dir="ltr">On Fri, Jul 5, 2013 at 4:27 PM, Garrett Smith <span dir="ltr"><<a href="mailto:g@rre.tt" target="_blank">g@rre.tt</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br></div>
At first glance, it looks like I'd be losing the connect timeout<br>
enhancements from here:<br>
<br>
<a href="https://github.com/gar1t/Emysql" target="_blank">https://github.com/gar1t/Emysql</a><br>
<br></blockquote><div> </div><div>Yes, we are currently missing those changes in the code base. I only took branches which were updated fairly recently on github, and yours were overlooked for some reason.</div><div><br>

</div><div>I like most of the changes you have in there and most of them definitely should be part of the full release. I also agree that your functional change to the gen_tcp:recv timeout is something we want to fix. The only slight worry is that it may be a too big behavioral change for some people. The other changes are changes I want as well, so i'll probably pull them unless you want to prepare a PR yourself.</div>

<div><br></div><div>I guess the initial reason for having the TCP timeout was to catch slow queries and get rid of them quickly, rather than waiting around. This makes a lot of sense in an environment where you have a large amount of small queries and are not allowed to wait around. But for larger queries and procedure calls, this is definitely not what one wants.</div>

<div><br></div><div>The whole decoder loop looks weird as well. It should be possible to make it smarter than what is it currently, but this is a future change.</div><div><br></div><div>Just drop me a note if you want to push it yourself or I should get to work. But it won't be this weekend, so I'll only be able to look at this Monday at the earliest, perhaps late Sunday.</div>

</div></div></div>