List:Commits« Previous MessageNext Message »
From:Alfranio Correia Date:May 7 2010 8:45am
Subject:Re: bzr commit into mysql-next-mr branch (aelkin:3140) Bug#50316
View as plain text  
Hi Andrei,


Andrei Elkin wrote:
> Alfranio, hello.
> 
>> Hi Andrei,
>>
>> The patch looks good and is straightforward.
>> I have just one question before approving it.
>>
>>
>> Andrei Elkin wrote:
>>> #At
> file:///home/andrei/MySQL/BZR/FIXES/next-mr-bug50316-Master_info_file-to-sss/ based on
> revid:alik@stripped
>>>
>>>  
>>>    if (mi->inited)
>>> @@ -169,7 +168,8 @@ int init_master_info(Master_info* mi, co
>>>  
>>>    mi->mysql=0;
>>>    mi->file_id=1;
>>> -  fn_format(fname, master_info_fname, mysql_data_home, "", 4+32);
>>> +  fn_format(mi->info_file_name, master_info_fname, mysql_data_home, "",
>>> +            MYF(MY_UNPACK_FILENAME|MY_RETURN_REAL_PATH|MY_SAFE_PATH));
>>>  
>> You changed also the flags in the fn_format.
>> Is there any impact on the change master? Can you check that?
> 
> My intent was actually to not let the slave to initialize master.info
> if the supplied --master-info-file=FILE_NAME would exceed the max allowed FN_REFLEN.
> But, you're right, there would be change in behaviour.
> 
> I am removing s/|MY_SAFE_PATH// to restore the original logics of shortening
> FILE_NAME to FN_REFLEN value in that case.

Sorry about that. Shame on me.
Please, keep the old patch and just check if the fn_format failed or not.

Cheers.
> 
> cheers,
> 
> &rei
Thread
bzr commit into mysql-next-mr branch (aelkin:3140) Bug#50316Andrei Elkin5 May
  • Re: bzr commit into mysql-next-mr branch (aelkin:3140) Bug#50316Alfranio Correia6 May
  • Re: bzr commit into mysql-next-mr branch (aelkin:3140) Bug#50316Luís Soares7 May
Re: bzr commit into mysql-next-mr branch (aelkin:3140) Bug#50316Alfranio Correia7 May