phorge-phorge/resources/sql/patches/20130621.diffcommentphid.sql at master - revi-archive/phorge-phorge - SiliconForest Atelier

revi-archive/phorge-phorge

mirror of https://we.phorge.it/source/phorge.git synced 2024-09-19 16:58:48 +02:00

epriestley 75fa580f3f Add PHIDs to DifferentialComments

Summary:
Ref T2222. This adds PHIDs to all Differential comments so I can migrate the inlinecommment table to transaction_comment in the next diff.

@wez, this will issue a few million queries for Facebook (roughly, one for each Differential comment ever made). It's safe to skip the `.php` half of the patch, bring Phabricator up normally, and then apply this patch with Phabricator running if that eases the migration, although the next few diffs will probably be downtime-required migrations so maybe it's easier to just schedule some downtime.

Test Plan: Ran migration locally. Verified existing comments and new comments received PHIDs.

Reviewers: btrahan

Reviewed By: btrahan

CC: wez, aran

Maniphest Tasks: T2222

Differential Revision: https://secure.phabricator.com/D6266

2013-06-21 18:41:14 -07:00

5 lines

198 B

SQL

Raw Permalink Blame History

 ALTER TABLE {$NAMESPACE}_differential.differential_comment
   ADD phid VARCHAR(64) NOT NULL COLLATE utf8_bin;
 ALTER TABLE {$NAMESPACE}_differential.differential_comment
   ADD KEY `key_phid` (phid);