Page MenuHomePhabricator

Remove the "search_documentfield" table
ClosedPublic

Authored by epriestley on Thu, May 23, 4:55 PM.

Details

Summary

Ref T11741. See PHI1276. After the switch to "Ferret", this table has no remaining readers or writers.

Test Plan
  • Ran bin/storage upgrade -f, no warnings.
  • Grepped for class name, table name, stemmedCorpus column; got no relevant hits.
  • Did a fulltext search.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Thu, May 23, 4:55 PM
Owners added a subscriber: Restricted Owners Package.Thu, May 23, 4:55 PM
epriestley requested review of this revision.Thu, May 23, 4:56 PM
amckinley accepted this revision.Thu, May 23, 5:18 PM
This revision is now accepted and ready to land.Thu, May 23, 5:18 PM

If we're trying to drop the MyISAM dependency, don't forget to update the quickstart when we're finished: https://secure.phabricator.com/source/phabricator/browse/master/resources/sql/quickstart.sql$9727

Yeah, the end goal is to get those two MyISAM lines out of quickstart.sql. This gets one of 'em but is probably less than half the work. ๐Ÿ™ˆ

This revision was automatically updated to reflect the committed changes.