Page MenuHomePhabricator

Stop writing to gbw_address and gbw_target_central_id
Closed, ResolvedPublic

Description

So that we can support global autoblocks, the global_block_whitelist table needs to no longer have columns for the target of the block. This is because we cannot appropriately hide this data in the wiki replicas, as the store of which block is an autoblock is stored on another database.

We can safely not write to the column as it is only used in places where we could instead write to the gbw_id column to be the reference to the target (through a separate DB query to the globalblocks table).

Event Timeline

Change #1073466 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@master] Add a default value to gbw_address

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

Dreamy_Jazz added projects: Schema-change, DBA.

Requesting DBA feedback on the attached patch. Would appreciate if this was done soon as this is a blocker.

ABran-WMF triaged this task as Medium priority.Sep 19 2024, 7:00 AM
ABran-WMF moved this task from Triage to Pending comment on the DBA board.
ABran-WMF added subscribers: Ladsgroup, Marostegui.

Change #1073466 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@master] Add a default value to gbw_address

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

The patch was merged. If there are issues with the proposed approach, these could probably be discussed in the Schema-change-in-production ticket T375184.

Given that T375184 is marked as blocked, I propose that we proceed by writing an empty string to the column to avoid blocking T374853.

Change #1075018 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@master] Stop writing real data to gbw_address and gbw_target_central_id

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

As we only write empty strings to gbw_address, this is no longer a blocker for T374853. Further work on this task is blocked by T375184.

Change #1075018 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@master] Stop writing real data to gbw_address and gbw_target_central_id

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

Dreamy_Jazz claimed this task.

Change #1077455 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/GlobalBlocking@master] Stop writing to gbw_address and gbw_target_central_id

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

Change #1077455 merged by jenkins-bot:

[mediawiki/extensions/GlobalBlocking@master] Stop writing to gbw_address and gbw_target_central_id

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