Page MenuHomePhabricator
Differential D18544

Remove the fulltext "reconstructDocument()" method
ClosedPublic
Actions

Authored by epriestley on Sep 6 2017, 5:50 PM.
Tags
None
Referenced Files
F15573723: D18544.id44542.diff
Mon, May 5, 9:41 PM
F15564233: D18544.id44548.diff
Wed, Apr 30, 11:12 AM
F15541917: D18544.id.diff
Fri, Apr 25, 2:22 PM
F15537935: D18544.diff
Thu, Apr 24, 4:44 PM
F15536947: D18544.id44548.diff
Thu, Apr 24, 11:28 AM
F15521972: D18544.id44548.diff
Sun, Apr 20, 5:05 PM
F15517258: D18544.diff
Sat, Apr 19, 1:09 AM
F15498434: D18544.id44548.diff
Sun, Apr 13, 12:57 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12819: InnoDB FULLTEXT appears to fail catastrophically once it reaches a moderate size
Commits
rPfaca1deea5b5: Remove the fulltext "reconstructDocument()" method
Summary

Ref T12819. This was originally intended for debugging, but never actually used and not clearly useful. There are no callers and it probably does not work. Just get rid of it.

Test Plan

Grepped for callers; none exist.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 6 2017, 5:50 PM
Harbormaster completed remote builds in B18325: Diff 44542.Sep 6 2017, 5:51 PM
chad accepted this revision.Sep 6 2017, 6:29 PM
This revision is now accepted and ready to land.Sep 6 2017, 6:29 PM
Closed by commit rPfaca1deea5b5: Remove the fulltext "reconstructDocument()" method (authored by epriestley). · Explain WhySep 6 2017, 6:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 44548

View Options

src/applications/search/fulltextstorage/PhabricatorElasticFulltextStorageEngine.php

Loading...
View Options

src/applications/search/fulltextstorage/PhabricatorFulltextStorageEngine.php

Loading...
View Options

src/applications/search/fulltextstorage/PhabricatorMySQLFulltextStorageEngine.php

Loading...
Phabricator · Built by Phacility