Page MenuHomePhabricator

Allow `bin/storage adjust` to adjust table engines
ClosedPublic

Authored by epriestley on Nov 24 2016, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 12, 7:44 PM
Unknown Object (File)
Mon, Feb 10, 1:34 PM
Unknown Object (File)
Mon, Feb 10, 1:34 PM
Unknown Object (File)
Mon, Feb 10, 1:34 PM
Unknown Object (File)
Sat, Feb 8, 12:46 AM
Unknown Object (File)
Mon, Feb 3, 9:18 AM
Unknown Object (File)
Wed, Jan 29, 6:59 AM
Unknown Object (File)
Sat, Jan 25, 3:08 PM
Subscribers
None

Details

Summary

Ref T11741. On recent-enough versions of MySQL, we would prefer to use InnoDB for fulltext indexes instead of MyISAM.

Allow bin/storage adjust to read actual and expected table engines, and apply adjustments as necessary.

We have one existing bad table that uses the wrong engine, metamta_applicationemail. This change corrects that table.

Test Plan
  • Ran bin/storage upgrade.
  • Saw the adjustment phase apply this change properly:
>>>[463] <query> ALTER TABLE `local_metamta`.`metamta_applicationemail` COLLATE = 'utf8mb4_bin', ENGINE = 'InnoDB'

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow `bin/storage adjust` to adjust table engines.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 24 2016, 5:18 PM
This revision was automatically updated to reflect the committed changes.