Page MenuHomePhabricator

[L] Remove WDQS layer in ApiRelatedConcepts
Closed, ResolvedPublic

Description

The current implementation for concept chips is using WDQS to traverse (and format) related statements.
This does not scale well, and must be removed before a large audience starts to consume the API.

This essentially means these changes in ApiRelatedConcepts::getTermsRelatedToWikibaseItems:

  • instead of traversing entity statements via WDQS, we'll only be able use the statements of the matched entities
  • this means that we'll lose some heuristics
  • heuristics config will likely have to be updated to match above new reality
  • and we'll have to add a bit of code to format the statement values (WDQS results already did that, until now)

Event Timeline

CBogen renamed this task from Remove WDQS layer in ApiRelatedConcepts to [L] Remove WDQS layer in ApiRelatedConcepts.Jan 13 2021, 5:27 PM

Change 658588 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[mediawiki/extensions/WikibaseMediaInfo@master] Alternative, simpler, heuristics-based concept chips implementation

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

Change 658589 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[operations/mediawiki-config@master] [WikibaseMediaInfo] MediaSearch: new set of heuristics for alternative implementation

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

Change 658590 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[operations/mediawiki-config@master] [WikibaseMediaInfo] MediaSearch: remove old, unused set of heuristics

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

Change 658589 merged by jenkins-bot:
[operations/mediawiki-config@master] [WikibaseMediaInfo] MediaSearch: new set of heuristics for alternative implementation

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

Change 658588 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Alternative, simpler, heuristics-based concept chips implementation

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

Etonkovidova subscribed.

Checked commons wmf.30 - found no regression issues.

Change 658590 abandoned by Matthias Mullie:

[operations/mediawiki-config@master] [WikibaseMediaInfo] MediaSearch: remove old, unused set of heuristics

Reason:

This got done in I3bebef09716b8480fad59d39de3bab7199614a77

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