List:Internals« Previous MessageNext Message »
From:Eric Bergen Date:May 19 2008 4:33pm
Subject:Re: Adding new flush commands
View as plain text  
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 <serg@stripped> 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üssen,
> Sergei
>
> --
>   __  ___     ___ ____  __
>  /  |/  /_ __/ __/ __ \/ /   Sergei Golubchik <serg@stripped>
>  / /|_/ / // /\ \/ /_/ / /__  Principal Software Engineer/Server Architect
> /_/  /_/\_, /___/\___\_\___/  Sun Microsystems GmbH, HRB München 161028
>       <___/                  Sonnenallee 1, 85551 Kirchheim-Heimstetten
> Geschäftsführer: Thomas Schroeder, Wolfgang Engels, Dr. Roland Boemer
> Vorsitzender des Aufsichtsrates: Martin Häring
>



-- 
high performance mysql consulting.
http://provenscaling.com
Thread
Adding new flush commandsEric Bergen13 May
  • Re: Adding new flush commandsSergei Golubchik13 May
    • RE: Adding new flush commandsRick James13 May
      • Re: Adding new flush commandsEric Bergen15 May
        • RE: Adding new flush commandsRick James15 May
          • Re: Adding new flush commandsEric Bergen15 May
            • RE: Adding new flush commandsRick James15 May
  • RE: Adding new flush commandsRick James15 May
    • Re: Adding new flush commandsEric Bergen15 May
    • Re: Adding new flush commandsMark Callaghan16 May
      • Re: Adding new flush commandsEric Bergen19 May
        • Re: Adding new flush commandsSergei Golubchik19 May
          • Re: Adding new flush commandsEric Bergen19 May
            • Re: Adding new flush commandsEric Bergen22 May
              • Re: Adding new flush commandsEric Bergen25 Oct