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
F2186900: D25430.id1365.diff
Fri, May 10, 18:02
F2186881: D25430.id1362.diff
Fri, May 10, 17:19
F2186771: D25430.id1363.diff
Fri, May 10, 15:08
Unknown Object (File)
Sun, May 5, 01:14
Unknown Object (File)
Thu, May 2, 13:00
Unknown Object (File)
Wed, May 1, 19:14
Unknown Object (File)
Wed, May 1, 19:14
Unknown Object (File)
Wed, May 1, 17:02
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