Page MenuHomePhabricator

preloading using multiple properties does not work
Closed, ResolvedPublic


interwiki uses the templates=True and langlinks=True functionality provided by APISite.preloadpages

site.preloadpages(pageGroup, templates=True, langlinks=True)

However, it does not work. Continuation for langlinks & templates fails. Only the revisions continuation is performed.

I presume it has never worked, as the commit that added it did not do anything related to handling of multiple continuations

Version: core-(2.0)
Severity: major



Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 4:00 AM
bzimport set Reference to bz73461.
bzimport added a subscriber: Unknown Object (????).
jayvdb created this task.Nov 15 2014, 11:40 AM

Change 173499 had a related patch set uploaded by John Vandenberg:
Preloading tests

As far as I can see, only interwiki uses this.
This bug is not causing errors, but langlinks & templates are not being preloaded, so interwiki is operating much slower than it would in compat.

Mpaa added a comment.Nov 15 2014, 2:46 PM

Support for multiple continuation has been added here:

Change 173499 merged by jenkins-bot:
Preloading tests

A quick update: improved the efficiency of the query generator underneath preloading , so much that it actually removed (hid) some of the preloading bugs.

test_preload_templates is consistently passing on en.wp, test.wd, fr.wikt, however it occasionally fails on ar.wp; usually all platforms fail together. e.g.

I'm not able to deduce what might be causing those irregular failures; my only (and probably wrong) guess is that the test is finding pages on ar.wp that dont have any templates.

Xqt closed this task as Resolved.May 12 2019, 12:32 PM
Xqt claimed this task.
Xqt added a subscriber: Xqt.

Tests passes