Hi, Sergei! 02.04.2019, 20:55, "Sergei Golubchik" <s...@mariadb.org>: > 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.
Done. Sorry I didn't noticed that for myself. > > Regards, > Sergei > Chief Architect MariaDB > and secur...@mariadb.org -- Eugene _______________________________________________ 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