Differential D10258

Provide a setup warning about using the default MySQL stopword file
ClosedPublic
Actions

Authored by epriestley on Aug 13 2014, 8:46 PM.

Tags

None

Referenced Files

	F13219971: D10258.id24701.diff
	Sat, May 18, 10:23 PM

	F13182003: D10258.id24694.diff
	Thu, May 9, 5:55 PM

	Unknown Object (File)
	May 5 2024, 6:20 AM

	Unknown Object (File)
	Apr 28 2024, 3:15 AM

	Unknown Object (File)
	Apr 24 2024, 11:38 AM

	Unknown Object (File)
	Apr 12 2024, 6:33 AM

	Unknown Object (File)
	Apr 4 2024, 12:40 PM

	Unknown Object (File)
	Mar 24 2024, 1:17 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T2605: Search on secure.phabricator.com doesn't find MySQL stopwords

Commits

Restricted Diffusion Commit
rPe616f166ae9f: Provide a setup warning about using the default MySQL stopword file

Summary

Fixes T2605.

Add a setup warning about the stopword file.
Provide a simpler stopword file.

Test Plan

Hit setup warning.
Resolved it according to instructions.
Added "various" to a task, then searched for it, found the task.

Diff Detail

Repository

Branch

stopword

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2214
Build 2218: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24694.Aug 13 2014, 8:46 PM

epriestley retitled this revision from to Provide a setup warning about using the default MySQL stopword file.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T2605: Search on secure.phabricator.com doesn't find MySQL stopwords.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 13 2014, 8:46 PM

epriestley added inline comments.Aug 13 2014, 8:47 PM

resources/sql/stopwords.txt
1	I generated this by taking a list of the 500 most common english words from Google and going down them randomly removing things I imagined people might conceivably want to search for.

Harbormaster completed remote builds in B2214: Diff 24694.Aug 13 2014, 8:47 PM

btrahan accepted this revision.Aug 13 2014, 9:56 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 13 2014, 9:56 PM

Closed by commit rPe616f166ae9f (authored by @epriestley).

chad mentioned this in T6539: Query: Searching for tasks which contain the word "do" does not seem to work.Nov 12 2014, 7:03 PM

epriestley mentioned this in D10955: Properly create Elasticsearch index.Dec 15 2014, 12:20 PM

Revision Contents
Changeset List

Path

Size

resources/

sql/

50 lines

src/

applications/

config/

check/

PhabricatorSetupCheckMySQL.php

52 lines

search/

selector/

PhabricatorDefaultSearchEngineSelector.php

9 lines

Diff 24694

resources/sql/stopwords.txt

Loading...

src/applications/config/check/PhabricatorSetupCheckMySQL.php

Loading...

src/applications/search/selector/PhabricatorDefaultSearchEngineSelector.php

Loading...