Hi, Eugene!

On Apr 02, Eugene Kosov wrote:
> Hi, Sergei.
> 
> Thank you for the review. Style fixed.
> 
> 01.04.2019, 17:18, "Sergei Golubchik" <s...@mariadb.org>:
> > Hi, Eugene!
> >
> > See the review below.
> > This looked pretty much ok, a couple of style comments.
> >
> > And why did you not implement the new ALTER TABLE .. .RENAME INDEX syntax?
> 
> Well, it's a different issue MDEV-7318 which is not assigned to me. I
> wasn't initiative enough to implement it too.
> I may do it if you want but I would like to do something related to
> InnoDB instead.

Okay. With the latest style fixes this PR looks fine.

But, please, rebase it on top of the latest 10.4. It causes numerous
merge conflicts with your "remove dead code" commit.

Regards,
Sergei
Chief Architect MariaDB
and secur...@mariadb.org

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : maria-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to