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

Assigned To

Authored By

	• Spage
	May 5 2015, 10:54 AM

Description

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.

Details

	Subject	Repo	Branch	Lines +/-
	Simplify search logic for performance benefit	mediawiki/extensions/MobileFrontend	master	+17 -117

Customize query in gerrit

Related Objects

Mentioned Here: T75522: PrefixSearch should support paging

Event Timeline

• Spage created this task.May 5 2015, 10:54 AM

• Spage raised the priority of this task from to Needs Triage.

• Spage updated the task description. (Show Details)

• Spage added projects: MobileFrontend, Performance Issue.

• Spage added subscribers: • Spage, MaxSem, Anomie.

Restricted Application added a project: Web-Team-Backlog. · View Herald TranscriptMay 5 2015, 10:54 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

• KLans_WMF moved this task from Incoming to 2014-15 Q4 on the Web-Team-Backlog board.May 6 2015, 5:15 PM

• Jhernandez triaged this task as Medium priority.Aug 17 2015, 11:19 AM

• Jhernandez removed a project: Web-Team-Backlog.Aug 17 2015, 11:25 AM

Jdlrobson added a project: Reading-Web-Sprint-67-If, Then, Else...?.Feb 17 2016, 10:01 PM

Jdlrobson set Security to None.

Jdlrobson added a project: good first task.Feb 29 2016, 5:44 PM

Jdlrobson moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-67-If, Then, Else...? board.

Jdlrobson moved this task from To Do to Doing on the Reading-Web-Sprint-67-If, Then, Else...? board.Mar 9 2016, 11:38 PM

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

https://gerrit.wikimedia.org/r/276384

gerritbot added a project: Patch-For-Review.Mar 10 2016, 12:02 AM

MaxSem unsubscribed.Mar 10 2016, 1:27 AM

Jdlrobson claimed this task.Mar 11 2016, 9:27 PM

Jdlrobson moved this task from Doing to Code Review on the Reading-Web-Sprint-67-If, Then, Else...? board.Mar 11 2016, 10:09 PM

Reviewing…

phuedx moved this task from Code Review to -1 (Needs More Work) on the Reading-Web-Sprint-67-If, Then, Else...? board.Mar 14 2016, 11:42 AM

Jdlrobson moved this task from -1 (Needs More Work) to Code Review on the Reading-Web-Sprint-67-If, Then, Else...? board.Mar 14 2016, 4:00 PM

Jdlrobson edited projects, added Reading-Web-Sprint-69-K-Phab-and-the-Kurious-Koders-mmKay; removed Reading-Web-Sprint-67-If, Then, Else...?.Mar 14 2016, 4:08 PM

Jdlrobson moved this task from Needs Analysis to Code Review on the Reading-Web-Sprint-69-K-Phab-and-the-Kurious-Koders-mmKay board.Mar 16 2016, 9:50 PM

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

https://gerrit.wikimedia.org/r/276384

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-03-22_(1.27.0-wmf.18)).Mar 17 2016, 4:02 PM

Jdlrobson closed this task as Resolved.Mar 17 2016, 5:15 PM

Jdlrobson removed a project: Reading-Web-Sprint-69-K-Phab-and-the-Kurious-Koders-mmKay.

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptAug 7 2016, 6:15 PM

Aklapper removed a subscriber: Anomie.Oct 16 2020, 5:42 PM

performance: MobileFrontend SearchApi.js query can drop list=prefixsearch and use index to sortClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

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