Paths

Table of Contentst

Differential D16940

Create and populate a stopwords table for InnoDB fulltext indexes to use in the future
ClosedPublic
Actions

Authored by epriestley on Nov 24 2016, 4:43 PM.

Tags

None

Referenced Files

	F15423724: D16940.id40784.diff
	Sat, Mar 22, 5:02 PM

	F15421610: D16940.id40773.diff
	Sat, Mar 22, 1:19 AM

	F15390086: D16940.diff
	Sat, Mar 15, 5:53 AM

	F15390085: D16940.id.diff
	Sat, Mar 15, 5:53 AM

	F15382100: D16940.id40773.diff
	Fri, Mar 14, 10:40 AM

	F15338795: D16940.diff
	Sun, Mar 9, 12:58 PM

	Unknown Object (File)
	Feb 18 2025, 12:06 PM

	Unknown Object (File)
	Feb 10 2025, 1:34 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud)

Commits

rPff3333548f8a: Create and populate a stopwords table for InnoDB fulltext indexes to use in the…

Summary

Ref T11741. InnoDB uses a stopwords table instead of a stopwords file.

During storage upgrade, synchronize the table from the stopwords file on disk.

Test Plan

Ran storage upgrade.
Ran select * from stopwords, saw stopwords.
Added some garbage to the table.
Ran storage upgrade, saw it remove it.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40773.Nov 24 2016, 4:43 PM

epriestley retitled this revision from to Create and populate a stopwords table for InnoDB fulltext indexes to use in the future.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a task: T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud).

We don't actually end up really needing this since the InnoDB stopwords are pretty reasonable, but I wrote it and it could make things easier in the future (e.g., it lets us tell users to change one setting to enable it). If it creates any maintenance headaches I'll just throw it out.

chad accepted this revision.Nov 24 2016, 6:53 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Nov 24 2016, 6:53 PM

Closed by commit rPff3333548f8a: Create and populate a stopwords table for InnoDB fulltext indexes to use in the… (authored by epriestley, committed by epriestley). · Explain WhyNov 25 2016, 11:13 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20161124.search.01.stopwords.sql

3 lines

src/

applications/

search/

storage/

PhabricatorSearchSchemaSpec.php

8 lines

document/

PhabricatorSearchDocument.php

2 lines

infrastructure/

storage/

management/

workflow/

PhabricatorStorageManagementUpgradeWorkflow.php

53 lines

Diff 40784

resources/sql/autopatches/20161124.search.01.stopwords.sql

Loading...

src/applications/search/storage/PhabricatorSearchSchemaSpec.php

Loading...

src/applications/search/storage/document/PhabricatorSearchDocument.php

Loading...

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementUpgradeWorkflow.php

Loading...