Page MenuHomePhabricator

Prefetch entities in fed props for pages using summary formatting
Closed, ResolvedPublic

Description

Although we already have a prefetching service ApiEntityLookup it is only used in a prefetching term lookup currently.
The hook that we use for formatting summaries only knows about 1 summary at a time and thus can not do prefetching.

(This relates slightly to T257912 which also tries to have some hook run on these pages at a higher level)
The hook may end up being the same as the one there, but may also be different.

When one of these pages is loaded (where this hook runs)
As few API calls as possible should be performed
By using the ApiEntityLookup to pre fetch and batch the entity lookups

Event Timeline

Change 620889 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] FP: WIP DNM prefetch history / recent changes

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

Change 620901 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] FP: Make entityLookup in ApiServiceFactory singleton

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

Change 621202 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] FP: Add prefetching of properties on recentchanges

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

Change 622153 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] FP: Add prefetching helper for summaries

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

Change 622572 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] FP: Add prefetching on diff-pages

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

Change 620901 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] FP: Make entityLookup in ApiServiceFactory singleton

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

Change 622153 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] FP: Add prefetching helper for summaries

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

Change 620889 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] FP: Add property prefetching on history pages

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

Change 622572 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] FP: Add prefetching on diff-pages

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

Change 621202 merged by Tobias Andersson:
[mediawiki/extensions/Wikibase@master] FP: Add prefetching of properties on recentchanges

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

Change 666894 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@REL1_35] Backport five FP changes

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

Change 666894 abandoned by Tobias Andersson:
[mediawiki/extensions/Wikibase@REL1_35] Backport five FP changes

Reason:

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

Change 667163 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@REL1_35] Backport five FP changes

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

Change 667163 abandoned by Tobias Andersson:
[mediawiki/extensions/Wikibase@REL1_35] Backport five FP changes

Reason:

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