Page MenuHomePhabricator

Rename "SearchEngine" to "FulltextStorageEngine"
ClosedPublic

Authored by epriestley on Dec 21 2015, 6:29 PM.
Tags
None
Referenced Files
F18085516: D14843.id.diff
Tue, Aug 5, 7:16 PM
F17937203: D14843.id35876.diff
Thu, Jul 31, 12:57 AM
F17751669: D14843.diff
Tue, Jul 22, 5:54 AM
F17738536: D14843.diff
Sun, Jul 20, 11:58 PM
Unknown Object (File)
Jun 16 2025, 2:36 PM
Unknown Object (File)
Jun 15 2025, 4:48 PM
Unknown Object (File)
May 25 2025, 6:46 AM
Unknown Object (File)
May 10 2025, 1:17 AM
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.