Thank you for the comments and the approval to push Kostja!
Narayanan
Konstantin Osipov wrote:
> * V Narayanan <V.Narayanan@stripped> [09/02/11 15:48]:
>
>> + /*
>> + Open and attaches the MyISAM tables,that are under the MERGE table
>> + parent, on the MyISAM storage engine interface directly within the
>> + MERGE engine. The new MyISAM table instances, as well as the MERGE
>> + clone itself, are not visible in the table cache. This is not a
>> + problem because all locking is handled by the original MERGE table
>> + from which this is cloned of.
>> + */
>>
>
> Please align the comment to follow the coding style:
>
>
>> + /*
>> + Open and attaches the MyISAM tables,that are under the MERGE table
>> + parent, on the MyISAM storage engine interface directly within the
>> + MERGE engine. The new MyISAM table instances, as well as the MERGE
>> + clone itself, are not visible in the table cache. This is not a
>> + problem because all locking is handled by the original MERGE table
>> + from which this is cloned of.
>> + */
>>
>
> <cut>
>
>
>> +/**
>> + * Returns a cloned instance of the current handler.
>> + *
>> + * @return A cloned handler instance.
>> + */
>>
>
> Correct alignment is:
>
>
>> +/**
>> + Returns a cloned instance of the current handler.
>> +
>> + @return A cloned handler instance.
>> +*/
>>
>
> OK to push.
>
>
