Page MenuHomePhabricator

Performance IssueTag
ActivePublic

Members (4)

Watchers (3)

Details

Description

Generic tag for current issues that are causing slow server responses or slow/costly client-side payloads.

This is a form of Technical-Debt (use that workboard).

(For the workboard of the Performance Team, see Performance-Team.)

Recent Activity

Yesterday

Maintenance_bot added a project to T235011: API query times out: Commons.
Thu, Dec 26, 7:30 PM · Commons, MediaWiki-File-management, Platform Engineering (Icebox), Performance Issue, MediaWiki-Action-API
Pppery added a project to T235011: API query times out: MediaWiki-File-management.
Thu, Dec 26, 6:47 PM · Commons, MediaWiki-File-management, Platform Engineering (Icebox), Performance Issue, MediaWiki-Action-API

Mon, Dec 23

TheDJ moved T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI from Inbox to For-review on the User-TheDJ board.
Mon, Dec 23, 8:12 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
TheDJ added a project to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI: User-TheDJ.
Mon, Dec 23, 8:12 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)

Sun, Dec 22

Nintendofan885 updated subscribers of T325062: Database error when trying to view user logs of user with many log events.

That looks for a specifc log type and is not affected, only viewing the default public logs.

Sun, Dec 22, 11:50 AM · Wikimedia-Slow-DB-Query, Wikimedia-production-error, Performance Issue, MediaWiki-Logevents

Wed, Dec 18

Daimona created T382436: $wgDebugToolbar is super slow because it stringifies the same messages over and over again.
Wed, Dec 18, 6:53 PM · Performance Issue, MediaWiki-Debug-Logger
Jdforrester-WMF moved T379977: Skip validation of instances of Wikidata types, for performance from Essential work to In Engineering on the Abstract Wikipedia team (25Q2 (Oct–Dec)) board.
Wed, Dec 18, 5:44 PM · Abstract Wikipedia team (25Q2 (Oct–Dec)), Abstract Wikipedia Fix-It tasks, Performance Issue, function-orchestrator
Joe added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

I don't see how we can stop caching and purging the rest api urls.

We are currently not using the CDN cache for the new page/html endpoints. This may however change when they get used more.

In general, the access pattern for these APIs is much more "flat" than organic access, since they tend to be used bycrawlers. So they benefit a lot less from caching, the hit rate is likely low.

Wed, Dec 18, 10:23 AM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
Xqt removed a project from T379482: Increase the common passwords list to 1M entries: Patch-For-Review.
Wed, Dec 18, 9:17 AM · Performance Issue, CommonPasswords
Xqt added a comment to T379482: Increase the common passwords list to 1M entries.

Change #1105275 had a related patch set uploaded (by Xqt; author: Dorna):

Sorry, wrong Task.

Wed, Dec 18, 9:17 AM · Performance Issue, CommonPasswords
gerritbot added a comment to T379482: Increase the common passwords list to 1M entries.

Change #1105275 had a related patch set uploaded (by Xqt; author: Dorna):

[pywikibot/core@master] Fix: Implement ISBN retrieval and handle missing data

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

Wed, Dec 18, 8:57 AM · Performance Issue, CommonPasswords
Legoktm added a comment to T382044: Use static arrays for ExtensionRegistry::loadFromQueue caching.

Time is a flat circle, back to loading extension information from PHP it seems :) This was proposed as part of the original RfC even:

Reading and parsing a JSON file will be slower than loading a PHP file due to APC and other factors. However, we can mitigate that by providing a script to "compile" all the required JSON files into their PHP equivalents.

Wed, Dec 18, 7:00 AM · MediaWiki-Platform-Team (Radar), Wikimedia-Performance-recommendation, Performance Issue, MediaWiki-Configuration

Tue, Dec 17

Michael moved T126486: Adding to Watchlist sometimes opens two additional windows from Inbox to Triaged on the Growth-Team board.
Tue, Dec 17, 7:02 PM · Growth-Team, Performance Issue, MediaWiki-Watchlist
daniel added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

I don't see how we can stop caching and purging the rest api urls.

Tue, Dec 17, 3:26 PM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
Joe added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

I don't see how we can stop caching and purging the rest api urls.

Tue, Dec 17, 2:56 PM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
Krinkle added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

Right now, we send purges for each edit as follows:

  • the main on-wiki url
  • the main on-wiki url with action=history
  • the mobile on-wiki url
  • the restbase url for page/html
Tue, Dec 17, 2:33 PM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
Aklapper lowered the priority of T360484: Submitting actions taking ~10 seconds to load after making changes from Medium to Low.
Tue, Dec 17, 2:31 PM · Release-Engineering-Team (Priority Backlog 📥), User-brennen, Performance Issue, Phabricator
Krinkle added a parent task for T382044: Use static arrays for ExtensionRegistry::loadFromQueue caching: T212460: Adopt static array files for local disk storage of values (epic).
Tue, Dec 17, 2:26 PM · MediaWiki-Platform-Team (Radar), Wikimedia-Performance-recommendation, Performance Issue, MediaWiki-Configuration
Krinkle added projects to T382044: Use static arrays for ExtensionRegistry::loadFromQueue caching: Wikimedia-Performance-recommendation, MediaWiki-Platform-Team (Radar).
Tue, Dec 17, 2:26 PM · MediaWiki-Platform-Team (Radar), Wikimedia-Performance-recommendation, Performance Issue, MediaWiki-Configuration

Mon, Dec 16

Lydia_Pintscher added a project to T374230: Frequent 500 Errors and Timeouts When Adding Statements to New Properties: Wikidata Dev Team.
Mon, Dec 16, 5:48 PM · Wikidata Dev Team, Performance Issue, Wikidata
Bugreporter added a comment to T374230: Frequent 500 Errors and Timeouts When Adding Statements to New Properties.

A workaround is install https://addons.mozilla.org/en-US/firefox/addon/wikimedia-debug-header/ and load (edit) this property using one of debug hosts.

Mon, Dec 16, 12:04 PM · Wikidata Dev Team, Performance Issue, Wikidata
Bugreporter added a project to T374230: Frequent 500 Errors and Timeouts When Adding Statements to New Properties: Performance Issue.
Mon, Dec 16, 11:50 AM · Wikidata Dev Team, Performance Issue, Wikidata

Fri, Dec 13

gerritbot added a project to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI: Patch-For-Review.
Fri, Dec 13, 10:03 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
gerritbot added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Change #1103502 had a related patch set uploaded (by TheDJ; author: TheDJ):

[mediawiki/extensions/Kartographer@master] Switch Kartographer fullscreen button to Codex

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

Fri, Dec 13, 10:03 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
TheDJ added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.
$content.find('.mw-kartographer-map[data-mw-kartographer], .mw-kartographer-map[data-mw="interface"]')
.each(function(index) {
    const container = this;
    const $container = $(container);
Fri, Dec 13, 9:39 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
TheDJ added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Wait a sec. Are we talking parsoid mode or old parser mode ?

Fri, Dec 13, 8:46 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
TheDJ added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Another mitigation approach to take here is to load the library when the map is scrolled into view. The leaflet library is also pretty big and loaded on page load.

Fri, Dec 13, 8:43 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
Jdlrobson added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Another mitigation approach to take here is to load the library when the map is scrolled into view. The leaflet library is also pretty big and loaded on page load.

Fri, Dec 13, 7:42 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
TheDJ added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Okay, I found that it's actually the dependency on mediawiki.router which depends in turn on oojs. This module provides OO.Router and uses OO.Registry.

Fri, Dec 13, 7:10 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
awight added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

Okay, I found that it's actually the dependency on mediawiki.router which depends in turn on oojs. This module provides OO.Router and uses OO.Registry.

Fri, Dec 13, 12:10 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
awight added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

The ResourceLoader modules already attempt to do exactly this, and the only direct OOUI dependency in static mode is currently oojs-ui.styles.icons-media which was not supposed to pull in the entire bundle. +1 to switching to codex styles.

Fri, Dec 13, 10:53 AM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
thiemowmde added a comment to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI.

For context: We worked on this extension in our past WMDE-GeoInfo-FocusArea. We are very interested but unfortunately can't make promisses. The issue appears to be quite old. I believe it goes back to 2016 when the static mode for high-traffic wikis was originally implemented, see T148070 and https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Kartographer/+/317018/8/modules/staticframe/staticframe.js#84. I guess the assumption back then was that OOUI will be loaded anyway on every page load. This is apparently not always true any more since we are slowly migrating to Codex and Desktop Improvements (Vector 2022).

Fri, Dec 13, 9:44 AM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
Bhsd updated the task description for T303232: Double braces syntax highlighting crash..
Fri, Dec 13, 2:53 AM · Community-Tech, MediaWiki-extensions-CodeMirror, Performance Issue, Browser-Support-Firefox

Thu, Dec 12

awight added a project to T381915: Adding a map to a page comes with performance penalty of loading entirety of OOUI: WMDE-TechWish-Maintenance.
Thu, Dec 12, 3:29 PM · User-TheDJ, Patch-For-Review, WMDE-TechWish-Maintenance, Performance Issue, Maps (Kartographer)
Stashbot added a comment to T54777: user_properties table bloat.

Mentioned in SAL (#wikimedia-operations) [2024-12-12T14:30:08Z] <Amir1> ladsgroup@mwmaint2002:~$ foreachwikiindblist all userOptions.php --delete VectorSkinVersion (T54777)

Thu, Dec 12, 2:30 PM · Wikimedia-maintenance-script-run, Performance Issue, WMF-General-or-Unknown
ItamarWMDE moved T170245: [REPO] Skip loading the property if the datatype has already been provided from Incoming to Repo on the wmde-wikidata-tech (DOT Triage) board.
Thu, Dec 12, 1:57 PM · wmde-wikidata-tech (DOT Triage), MediaWiki-extensions-PropertySuggester, MediaWiki-extensions-WikibaseRepository, Performance Issue, Wikidata
ItamarWMDE renamed T170245: [REPO] Skip loading the property if the datatype has already been provided from Skip loading the property if the datatype has already been provided to [REPO] Skip loading the property if the datatype has already been provided.
Thu, Dec 12, 1:56 PM · wmde-wikidata-tech (DOT Triage), MediaWiki-extensions-PropertySuggester, MediaWiki-extensions-WikibaseRepository, Performance Issue, Wikidata
ItamarWMDE edited projects for T170245: [REPO] Skip loading the property if the datatype has already been provided, added: wmde-wikidata-tech (DOT Triage); removed [DEPRECATED] wdwb-tech.
Thu, Dec 12, 10:55 AM · wmde-wikidata-tech (DOT Triage), MediaWiki-extensions-PropertySuggester, MediaWiki-extensions-WikibaseRepository, Performance Issue, Wikidata
Joe added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

For completeness, another option is the varnish "x-key" system, which involves two research projects. One is that implementation of x-key in varnish appears to be incomplete, and the second is that the assignment of appropriate x-keys to URLs is non-trivial as well. There are too many templates used on a page like [[Barack Obama]] to naively assign one x-key to every recursively-included template, so we still need to come up with a mechanism to determine which of the templates deserve an x-key assigned, likely based on purge statistics.

Thu, Dec 12, 9:12 AM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
daniel updated the task description for T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.
Thu, Dec 12, 8:31 AM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
Daimona created T382044: Use static arrays for ExtensionRegistry::loadFromQueue caching.
Thu, Dec 12, 3:23 AM · MediaWiki-Platform-Team (Radar), Wikimedia-Performance-recommendation, Performance Issue, MediaWiki-Configuration

Wed, Dec 11

Jdforrester-WMF moved T314580: Modify wikilambdafn_search endpoint to accept multiple types in wikilambdafn_type from Backlog to Engineering Backlog on the Abstract Wikipedia team board.
Wed, Dec 11, 8:25 PM · Performance Issue, Abstract Wikipedia Fix-It tasks, Abstract Wikipedia team, WikiLambda
Jdforrester-WMF moved T334236: Consider Also Serializing Evaluator Responses and Reentrant Calls with Avro from Backlog to Engineering Backlog on the Abstract Wikipedia team board.
Wed, Dec 11, 8:22 PM · Performance Issue, Abstract Wikipedia Fix-It tasks, function-evaluator, function-orchestrator, Abstract Wikipedia team
Jdforrester-WMF moved T313136: Minimize conversions between ZWrapper & JSON representations of ZObjects from Backlog to Engineering Backlog on the Abstract Wikipedia team board.
Wed, Dec 11, 8:16 PM · Abstract Wikipedia team, Performance Issue, Abstract Wikipedia Fix-It tasks, function-orchestrator
Jdlrobson added a comment to T381914: Growth experiments help icon loads OOjs UI on page load.

My primary concern here would be the article namespace, but generally given the increased adoption of Codex, OOUI on page load anywhere is probably not a good idea (particularly if that page is already loading Codex)

Wed, Dec 11, 6:57 PM · Performance Issue, Growth-Team, GrowthExperiments
Sgs moved T381914: Growth experiments help icon loads OOjs UI on page load from Inbox to Backlog on the Growth-Team board.
Wed, Dec 11, 6:25 PM · Performance Issue, Growth-Team, GrowthExperiments
Sgs added a comment to T381914: Growth experiments help icon loads OOjs UI on page load.

Is this considered a bug for all namespaces or for some in particular? cc @Jdlrobson-WMF

Wed, Dec 11, 6:25 PM · Performance Issue, Growth-Team, GrowthExperiments
Jdforrester-WMF triaged T380764: Consider eliminating remaining resolutions of Z11s and Z12s as Medium priority.
Wed, Dec 11, 6:14 PM · Performance Issue, Abstract Wikipedia team (25Q2 (Oct–Dec)), Abstract Wikipedia Fix-It tasks, function-orchestrator
Jdforrester-WMF moved T380764: Consider eliminating remaining resolutions of Z11s and Z12s from Incoming to Essential work on the Abstract Wikipedia team (25Q2 (Oct–Dec)) board.
Wed, Dec 11, 6:14 PM · Performance Issue, Abstract Wikipedia team (25Q2 (Oct–Dec)), Abstract Wikipedia Fix-It tasks, function-orchestrator
cscott added a comment to T369898: Reduce the number of resource_change and resource_purge events emitted due to template changes.

This might be a reasonable pair of hypotheses for annual planning:

  1. Instrument edits to identify edits with the highest rank (edit frequency * usage count * % no op edits) and analyze to determine edit reason and whether website scalability could be improved by omitting CDN purges for these edits.
Wed, Dec 11, 3:27 PM · Content-Transform-Team, Essential-Work, MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), serviceops, Performance Issue, MediaWiki-Engineering, MediaWiki-Core-HTTP-Cache, ChangeProp
  NODES
admin 1
COMMUNITY 1
Idea 1
idea 1
Note 8
Project 13