Homec4science

Migrate all Differential reviewer data to edges

Authored by epriestley <git@epriestley.com> on Oct 5 2013, 22:48.

Description

Migrate all Differential reviewer data to edges

Summary:
Ref T1279. @champo did a lot of this work already; we've been doing double writes for a long time.

Add "double reads" (reading the edge table as both the "relationship" table and as the "reviewer status" table), and migrate all the data.

I'm not bothering to try to recover old reviewer status (e.g., we could infer from transactions who accepted old revisions) because it wold be very complicated and doesn't seem too valuable.

Test Plan:

  • Without doing the migration, used Differential. Verified that reads and writes worked. Most of the data was there anyway since we've been double-writing.
  • Performed the migration. Verified that everything was still unchanged.
  • Dropped the edge table, verified all reviweer data vanished.
  • Migrated again, verified the reviewer stuff was restored.
  • Did various cc/reviewer/subscriber queries, got consistent results.

Reviewers: btrahan

Reviewed By: btrahan

CC: champo, aran

Maniphest Tasks: T1279

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

Details

Committed
epriestley <git@epriestley.com>Oct 5 2013, 22:54
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbd8071bb3350: Add a `(dst, type, src)` key to Differential's edge table
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH65ddefad8b9c: Migrate all Differential reviewer data to edges (authored by epriestley <git@epriestley.com>).Oct 5 2013, 22:54