Page MenuHomePhabricator

performance: MobileFrontend SearchApi.js query can drop list=prefixsearch and use index to sort
Closed, ResolvedPublic


I copied the query in SearchApi.js' getApiData() for an article. Brad pointed out you can leave out

list: 'prefixsearch',
pssearch: query,
pslimit: 15

from the query and instead sort the elements of query.pages on their index key added in T75522: PrefixSearch should support paging.

Event Timeline

Spage raised the priority of this task from to Needs Triage.
Spage updated the task description. (Show Details)
Spage added subscribers: Spage, MaxSem, Anomie.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 276384 had a related patch set uploaded (by Jdlrobson):
WIP: Cleanup use of prefixsearch in search

Change 276384 merged by jenkins-bot:
Simplify search logic for performance benefit