From: Eric Bergen Date: May 22 2008 2:56am Subject: Re: Adding new flush commands List-Archive: http://lists.mysql.com/internals/35621 Message-Id: <28fd48f00805211956u6d3e6269k9b035eb2f765a81d@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Incase you haven't seen it. I filed a bug for the parser issue: http://bugs.mysql.com/bug.php?id=3D36813 I also updated the patch to be more aligned with the mysql coding standards= . The new patch is at: http://ebergen.net/patches/flush_logs.patch On Mon, May 19, 2008 at 9:33 AM, Eric Bergen wrote= : > It's on the way. I was working on it last night and got distracted. > > -Eric > > On Sun, May 18, 2008 at 10:48 PM, Sergei Golubchik wrote= : >> Hi! >> >> On May 18, Eric Bergen wrote: >>> I updated the patch http://ebergen.net/patches/flush_logs.patch with >>> the suggestions from Sergei. It now supports log or logs and the log >>> function works. The patch applies against both 5.1 and 6.0 >>> >>> I found an interesting bug while doing the conversion. It seems that >>> functions that are special cased in the parser don't return invalid >>> number of argument errors correctly. For example: >>> >>> mysql> select left(1); >>> ERROR 1064 (42000): You have an error in your SQL syntax; check the >>> manual that corresponds to your MySQL server version for the right >>> syntax to use near ')' at line 1 >>> mysql> select locate(1); >>> ERROR 1582 (42000): Incorrect parameter count in the call to native >>> function 'locate' >>> >>> I'm not sure what the best way is to fix this. >> >> Eh, report a bug, perhaps ? >> >> Regards / Mit vielen Gr=FCssen, >> Sergei >> >> -- >> __ ___ ___ ____ __ >> / |/ /_ __/ __/ __ \/ / Sergei Golubchik >> / /|_/ / // /\ \/ /_/ / /__ Principal Software Engineer/Server Archite= ct >> /_/ /_/\_, /___/\___\_\___/ Sun Microsystems GmbH, HRB M=FCnchen 16102= 8 >> <___/ Sonnenallee 1, 85551 Kirchheim-Heimstetten >> Gesch=E4ftsf=FChrer: Thomas Schroeder, Wolfgang Engels, Dr. Roland Boeme= r >> Vorsitzender des Aufsichtsrates: Martin H=E4ring >> > > > > -- > high performance mysql consulting. > http://provenscaling.com > --=20 high performance mysql consulting. http://provenscaling.com