Page MenuHomePhabricator

Drop campaign_events.event_answers_status
Closed, ResolvedPublic

Description

The field was recently introduced in T341142 (still not fully applied to production DBs). However, we determined that the field isn't really necessary, because the data it stores can always be derived from other fields, and therefore it would only unnecessarily make the table larger. It would also have to be special-cased in the code due to its special meaning, which is one more reason in favour of removing it.

Checklist

  • Add default value again in beta (prod still has it)
  • Remove all mentions to the field from the code
  • Drop the column in the code
  • Drop the column in beta
  • Drop the column in prod (T342966)

Event Timeline

Mentioned in SAL (#wikimedia-releng) [2023-07-28T00:35:20Z] <Daimona> Adding default to campaign_events.ce_answers_status because the column will be dropped soon # T342939

Change 942531 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] Drop campaign_events.ce_answers_status

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

Change 942531 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Drop campaign_events.ce_answers_status

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

Mentioned in SAL (#wikimedia-releng) [2023-07-28T20:35:04Z] <Daimona> Dropping campaign_events.ce_answers_status in beta wikishared # T342939

Daimona updated the task description. (Show Details)