Page MenuHomePhorge

Implement ferret engine in typeahead datasource query for repos
ClosedPublic

Authored by bekay on Sep 2 2023, 16:24.
Tags
None
Referenced Files
F2948824: D25430.1738593030.diff
Sun, Feb 2, 14:30
F2947565: D25430.1738479576.diff
Sat, Feb 1, 06:59
F2945458: D25430.1738319809.diff
Thu, Jan 30, 10:36
F2945002: D25430.1738278962.diff
Wed, Jan 29, 23:16
F2942764: D25430.1738134343.diff
Tue, Jan 28, 07:05
F2942736: D25430.1738131703.diff
Tue, Jan 28, 06:21
F2941098: D25430.1738091150.diff
Mon, Jan 27, 19:05
F2929491: D25430.1737658339.diff
Wed, Jan 22, 18:52
Tokens
"Love" token, awarded by 20after4.

Details

Summary

This broadens the typeahead datasource search for repos. Before this patch a repository named "Alligator Simulator" would not be found with the search string "simu...". This is patched with the ferret engine search and indexing features.

See T15583

Test Plan

Create repositories with titles with 2 or more words. Search for these repos with the global typeahead search. The search term should begin with the second/third/n-th word of the repo title.

Unit TestsFailed

TimeTest
0 msPhabricatorCelerityTestCase::testCelerityMaps
EXCEPTION (RuntimeException): Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage' #0 [internal function]: PhutilErrorHandler::handleError(2, '...', '...', 703) #1 /var/www/phacility/arcanist/src/unit/engine/phutil/PhutilTestCase.php(703): xdebug_start_code_coverage(3)
0 msPhabricatorConduitTestCase::testConduitMethods
EXCEPTION (RuntimeException): Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage' #0 [internal function]: PhutilErrorHandler::handleError(2, '...', '...', 703) #1 /var/www/phacility/arcanist/src/unit/engine/phutil/PhutilTestCase.php(703): xdebug_start_code_coverage(3)
0 msPhabricatorInfrastructureTestCase::testApplicationsInstalled
EXCEPTION (RuntimeException): Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage' #0 [internal function]: PhutilErrorHandler::handleError(2, '...', '...', 703) #1 /var/www/phacility/arcanist/src/unit/engine/phutil/PhutilTestCase.php(703): xdebug_start_code_coverage(3)
0 msPhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries
EXCEPTION (RuntimeException): Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage' #0 [internal function]: PhutilErrorHandler::handleError(2, '...', '...', 703) #1 /var/www/phacility/arcanist/src/unit/engine/phutil/PhutilTestCase.php(703): xdebug_start_code_coverage(3)
0 msPhabricatorLibraryTestCase::testEverythingImplemented
EXCEPTION (RuntimeException): Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage' #0 [internal function]: PhutilErrorHandler::handleError(2, '...', '...', 703) #1 /var/www/phacility/arcanist/src/unit/engine/phutil/PhutilTestCase.php(703): xdebug_start_code_coverage(3)
View Full Test Results (7 Failed)

Event Timeline

bekay requested review of this revision.Sep 2 2023, 16:24

Add back code which shouldn't be removed.

This revision is now accepted and ready to land.Sep 3 2023, 06:41