Page MenuHomePhabricator

Create repo-aware TermSqlIndex
Closed, ResolvedPublic

Description

To allow searches to be performed on a foreign repo via direct DB access, we need TermSqlIndex to be aware not only of the DB name, but also of the repo name. This is needed to construct EntityIds with the correct repo prefix to be used in the TermIndexEntry objects returned by TermSqlIndex.

Implementation note: TermIndexEntry is currently bound closely to the actual table structure, and there is no obvious way to supply the repo name here. The problem is rooted in the fact that the terms table uses numeric entity IDs. Some refactoring may be needed to make this work nicely.

Related Objects

StatusSubtypeAssignedTask
OpenNone
OpenNone
DuplicateNone
OpenFeatureNone
OpenFeatureNone
DuplicateNone
ResolvedNone
ResolvedNone
ResolvedNone
OpenNone
OpenNone
StalledNone
ResolvedLydia_Pintscher
ResolvedLydia_Pintscher
ResolvedLydia_Pintscher
ResolvedLydia_Pintscher
ResolvedWMDE-leszek
ResolvedWMDE-leszek
DeclinedWMDE-leszek
ResolvedWMDE-leszek
ResolvedNone
ResolvedWMDE-leszek
ResolvedNone

Event Timeline

WMDE-leszek triaged this task as High priority.
WMDE-leszek moved this task from Proposed to Doing on the Wikidata-Former-Sprint-Board board.

Change 323846 had a related patch set uploaded (by WMDE-leszek):
[DNM]Make TermSqlIndex configured for a single repository only

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

Change 323846 merged by jenkins-bot:
Make TermSqlIndex configured for a single repository only

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