Last modified: 2014-02-10 04:35:41 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T51189, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49189 - New fields: ar_id, el_id.
New fields: ar_id, el_id.
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Highest normal (vote)
: ---
Assigned To: Sean Pringle
:
: 49198 (view as bug list)
Depends on:
Blocks: 49188 24538 49088 58435
  Show dependency treegraph
 
Reported: 2013-06-05 16:58 UTC by Greg Grossmeier
Modified: 2014-02-10 04:35 UTC (History)
14 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Greg Grossmeier 2013-06-05 16:58:20 UTC
Schema Change:

All wikis
tables: archive, externallinks, external_user (new) 
changes: New fields: ar_id, el_id.
status: Needs review
links: Gerrit change #51675, bug 15441, bug 39675
Comment 1 Greg Grossmeier 2013-06-08 20:26:16 UTC
Reedy: adding you on CC since you own the patch. Do you have a priority/importance for this schema change?
Comment 2 Sam Reed (reedy) 2013-06-27 19:46:16 UTC
(In reply to comment #1)
> Reedy: adding you on CC since you own the patch. Do you have a
> priority/importance for this schema change?

Not really. Long overdue more than anything.

Merging that patch wouldn't make any difference to core as it's not currently used. But likelihood that it would start being used properly, even more when there'
Comment 3 Sam Reed (reedy) 2013-06-27 20:23:21 UTC
(In reply to comment #2)
> Merging that patch wouldn't make any difference to core as it's not currently
> used. But likelihood that it would start being used properly, even more when
> there'

There's a script to back populate the ar_len field. To populate this easily/efficiently, this PK is really needed
Comment 4 Greg Grossmeier 2013-07-10 15:39:02 UTC
*** Bug 49198 has been marked as a duplicate of this bug. ***
Comment 5 Marc A. Pelletier 2013-10-30 14:24:22 UTC
As a reminder, replicated a redacted version of archive to labs is pending on the addition of the primary key on archive.
Comment 6 Sean Pringle 2013-10-31 08:26:11 UTC
ar_id has been added to all wikis except enwiki where it's still in progress. Another day or so required there as I have it well dialed back to avoid lagging slaves.

el_id change will begin afterwards.

It's slow going because the online alteration method I'm using is two-stage to compensate for the lack of primary and unique keys.

In hindsight altering these two tables online hasn't really saved much wall-clock time compared to altering tables on slaves and rotating masters, but it has nicely avoided any client disruption.
Comment 7 Yuvi Panda 2013-11-22 23:17:22 UTC
@Sean: Any update on this?
Comment 8 Sean Pringle 2013-11-25 04:49:15 UTC
el_id still remains to be added to several of the larger wikis, including enwiki. Other issues have put this process on (hopefully brief) hold.
Comment 9 Umherirrender 2014-01-15 19:54:19 UTC
Increase priority, because a change to master was merged, where the new field el_id is needed. To avoid fatal on wmf branch a fix is needed, see Gerrit change #105243, or the revert Gerrit change #107415 or bug 58435.
Comment 10 Sam Reed (reedy) 2014-01-15 20:11:14 UTC
Anyone got a time machine?

Sean, where are we up to now? I know domas killed one alter..
Comment 11 Sean Pringle 2014-01-16 05:56:48 UTC
Yes, one schema change job broke replication so the others were put on hold. The remaining wikis are:

enwiki
commonswiki
frwiki
itwiki
nlwiki
plwiki

I'll need to do master rotations for these.

Higher priority noted.
Comment 12 Sean Pringle 2014-02-10 04:35:41 UTC
This is now complete. All wikis have both ar_id and el_id.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links