Page MenuHomePhabricator

Rename br_user column
Closed, ResolvedPublic

Description

br_user VARCHAR(255) NOT NULL, -- Email Id of failing recipient

This column name/text are a bit confusing. It should probably be renamed. Depending on how widely this is deployed anywhere, it might not be worth the hassle, in which case, at least updating the comment to not say "email id" which suggests an integer id would be good


Version: unspecified
Severity: normal

Details

Reference
bz72683

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:44 AM
bzimport set Reference to bz72683.

Change 170101 had a related patch set uploaded by 01tonythomas:
Rename br_user column to br_user_email and update comments

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

Change 170101 merged by jenkins-bot:
Rename br_user column to br_user_email and update comments

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