From: Roy Lyseng Date: February 11 2011 2:29pm Subject: Re: bzr commit into mysql-trunk branch (martin.hansson:3272) WL#3724 List-Archive: http://lists.mysql.com/commits/131133 Message-Id: <4D5547C0.7030703@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 11.02.11 15.26, Martin Hansson wrote: > Roy Lyseng skrev 2011-02-11 14.46: >> Hi Martin, >> >> thank you for this patch. I have now revisited all code, and AFAIK things look >> good. I have some minor suggestions that you need to consider, though. > Oh no! You shouldn't have... ;-) As I told you on IRC the central algorithm is > all wrong. I finished a new patch this morning but I haven't committed because > Omer said in the conf call last night that we should not do anything that might > upset bugs.mysql.com today. Such as commits, apparently... Yes, I knew about that, but was so deeply into "scattering" a mail with comments that I decided to send them after all ;) > > Anyway, thank you for the comments, most of them seem independent of the changes > in the new patch so I'll incorporate them as well. And be ready for yet another > mid-size change next week! > > Have a nice weekend! Thanks, and the same to you! Roy