Page MenuHomePhabricator

Make Wikibase wb_terms term_search_key index the same as wb_terms_search_key in WMF production
Closed, ResolvedPublic5 Estimated Story Points

Description

T85414#4597677 details the current differences between the Wikibase DB schema as written in code and deployed to the WMF.

wb_terms_search_key is only in prod
term_search_key is created on install
term_search_key includes term_language, however wb_terms_search_key does not.

We need to

  • rename the term_search_key index to match the wb_terms_search_key used in production
  • remove the usage of term_language by the index

Priority & Impact
Having differing schema between WMF production and what we use elsewhere and in development causes outages like the tmp1 outage.
The tmp1 index caused a massive outage, we want to avoid doing that in the future.
This task is small and should be easy to complete so lets get it done and avoid it sitting here for a year, at which point we may have forgotten about the hardships that it caused.

Acceptance Criteria

  • Wikibase users upgrading from a version prior to this patch will have the index adjusted
  • Wikibase users installing a fresh install will have the correct index
  • The index should appear exactly as it does in WMF production

Event Timeline

Addshore triaged this task as Medium priority.Sep 19 2018, 3:50 PM
Addshore created this task.
Addshore moved this task from Incoming to Ready to estimate on the Wikidata-Campsite board.
jcrespo subscribed.

^I am assuming this doesn't have yet actionables for us (yet) just on the radar.

^I am assuming this doesn't have yet actionables for us (yet) just on the radar.

There will be no actionables for the DBAs on this ticket (or the other 2 that I just filed).
These are all bringing the in code schema in line with what is already in production.

Change 463302 had a related patch set uploaded (by Matthias Geisler; owner: Matthias Geisler):
[mediawiki/extensions/Wikibase@master] Changing term_text, term_search_key and wb_terms_entity_id indices

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

Change 463956 had a related patch set uploaded (by Matthias Geisler; owner: Matthias Geisler):
[mediawiki/extensions/Wikibase@master] Changing term_text, etc. whitespacefix

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

Change 463956 abandoned by Matthias Geisler:
Changing term_text, etc. whitespacefix

Reason:
Abandoned because this should be fixed by the previous patch and I it was just a misunderstanding of the usage of gerrit.

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

Change 463302 abandoned by WMDE-leszek:
Adjust wb_terms SQL indexes to match WMF production

Reason:
squashed into I152ca2233afbdc7a9c95b27eb5e649f61e901ede

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

Change 463302 restored by WMDE-leszek:
Adjust wb_terms SQL indexes to match WMF production

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

Addshore renamed this task from Make Wikibase wb_terms term_search_key index the same as wb_term_search_key in WMF production to Make Wikibase wb_terms term_search_key index the same as wb_terms_search_key in WMF production.Oct 10 2018, 8:48 AM
Addshore updated the task description. (Show Details)

Change 466900 had a related patch set uploaded (by WMDE-leszek; owner: WMDE-leszek):
[mediawiki/extensions/Wikibase@master] Replaced term_search_key SQL index with wb_terms_search_key

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

Change 463302 abandoned by WMDE-leszek:
Adjust wb_terms SQL indexes to match WMF production

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

Change 466900 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Replaced term_search_key SQL index with wb_terms_search_key

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

Everything now looks great on a fresh install

image.png (300×815 px, 62 KB)