Last modified: 2013-12-23 02:38:20 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 T60650, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58650 - Add user_ip fields to multiple flow tables on wmf's extension1db
Add user_ip fields to multiple flow tables on wmf's extension1db
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Sean Pringle
:
Depends on:
Blocks: 49188
  Show dependency treegraph
 
Reported: 2013-12-18 17:29 UTC by Erik Bernhardson
Modified: 2013-12-23 02:38 UTC (History)
0 users

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


Attachments

Description Erik Bernhardson 2013-12-18 17:29:21 UTC
Short description:

The current *_user_text fields are being replaced with nullable *_user_ip fields containing the user_ip only for anonymous users.  This first half of the schema update adds the *_user_ip fields.

Which wikis are affected:

test, test2, and mediawiki.  But the changes are only to tables 
on the extension1 cluster's flowdb.

Which tables:

flow_workflow
flow_revision
flow_tree_revision


What is the change to those tables:

ALTER TABLE /*_*/flow_workflow ADD workflow_user_ip varbinary(39) default null;
ALTER TABLE /*_*/flow_tree_revision ADD tree_orig_user_ip varbinary(39) default null;

ALTER TABLE /*_*/flow_revision
        ADD rev_user_ip varbinary(39) default null,
        ADD rev_mod_user_ip varbinary(39) default null,
        ADD rev_edit_user_ip varbinary(39) default null;

Links to gerrit changes and/or other related bug reports.

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

These new columns are not queried directly, only selected as part of other queries.   They are 39 bytes to hold either ipv4 or ipv6 addresses formatted as strings, same as they are displayed in the interface.  They are superseding the  current *_user_text fields which will be dropped from the table after migrations have run.
Comment 1 Sean Pringle 2013-12-23 02:38:20 UTC
Schema changes done.

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


Navigation
Links