Page MenuHomePhabricator

Standardise type for timestamp columns in FlaggedRevs extension
Closed, ResolvedPublic

Description

According to T42626 the timestamp should be adjust from varbinary(14) to binary(14) because there is no need to hold infinite.

Affected columns:
flaggedpages.fp_pending_since
flaggedpage_pending.fpp_pending_since
flaggedrevs.fr_rev_timestamp
flaggedrevs.fr_timestamp
flaggedrevs_statistics.frs_timestamp

On the following columns the default of empty string needs to be removed:
flaggedrevs.fr_rev_timestamp

On the following columns the default of infinity needs to be removed:
flaggedpage_config.fpc_expiry

Event Timeline

Change 804753 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/FlaggedRevs@master] Standardise type for timestamp columns

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

Change 804753 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] Standardise type for timestamp columns

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