Page MenuHomePhabricator

Rename "SearchEngine" to "FulltextStorageEngine"
ClosedPublic

Authored by epriestley on Dec 21 2015, 6:29 PM.
Tags
None
Referenced Files
F14045722: D14843.diff
Wed, Nov 13, 9:29 AM
F14024403: D14843.id35876.diff
Thu, Nov 7, 9:08 AM
F14024178: D14843.diff
Thu, Nov 7, 6:51 AM
F14013346: D14843.diff
Sat, Nov 2, 4:18 AM
F14012486: D14843.id.diff
Fri, Nov 1, 1:33 PM
F14012485: D14843.id35895.diff
Fri, Nov 1, 1:33 PM
F14012484: D14843.id35876.diff
Fri, Nov 1, 1:32 PM
F14012339: D14843.diff
Fri, Nov 1, 10:54 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
Branch
index7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9695
Build 11628: Run Core Tests
Build 11627: arc lint + arc unit

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.