Page MenuHomePhabricator

Rename "SearchEngine" to "FulltextStorageEngine"
ClosedPublic

Authored by epriestley on Dec 21 2015, 6:29 PM.
Tags
None
Referenced Files
F15451580: D14843.id35895.diff
Fri, Mar 28, 10:49 PM
F15447123: D14843.id35876.diff
Thu, Mar 27, 9:59 PM
F15440883: D14843.id.diff
Wed, Mar 26, 3:12 PM
F15435082: D14843.diff
Tue, Mar 25, 6:45 AM
F15389633: D14843.id.diff
Sat, Mar 15, 5:31 AM
F15387999: D14843.id35876.diff
Sat, Mar 15, 3:11 AM
F15387992: D14843.diff
Sat, Mar 15, 3:10 AM
F15378024: D14843.diff
Thu, Mar 13, 12:10 PM
Subscribers
None

Details

Summary

Ref T9979. I picked this name long before the advent of modern "Engine" architecture and it ended up being pretty confusing.

Rename "SearchEngine" (currently: mysql or elasticsearch, used to store and query fulltext indexes) to "FulltextStorageEngine" to make it more clear what it does and disambituate it from ApplicationSearch, which also has a bunch of stuff called "SearchEngine", "SearchEngineExtension", etc.

Test Plan

Grepped for phabricatorsearchengine.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Rename "SearchEngine" to "FulltextStorageEngine".
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.Dec 21 2015, 8:32 PM
This revision was automatically updated to reflect the committed changes.