Page MenuHomePhabricator

Update table schema for "AffectedPath" table
ClosedPublic

Authored by epriestley on Mar 15 2021, 10:37 PM.
Tags
None
Referenced Files
F15539401: D21617.id51464.diff
Fri, Apr 25, 1:28 AM
F15538852: D21617.id51464.diff
Thu, Apr 24, 9:52 PM
F15538353: D21617.id51453.diff
Thu, Apr 24, 6:46 PM
F15532649: D21617.id.diff
Wed, Apr 23, 4:50 PM
F15527219: D21617.diff
Tue, Apr 22, 1:58 AM
F15515786: D21617.id51464.diff
Fri, Apr 18, 1:23 PM
F15508699: D21617.diff
Wed, Apr 16, 6:11 AM
F15466882: D21617.diff
Thu, Apr 3, 10:34 AM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13639. Make schema changes:

  • Make repositoryID nullable, for revisions with no repository.
  • Remove "epoch", which has no readers and no clear use.
  • Change the ordering of the key, since "pathID" has more unique values and no queries ever issue without it.
Test Plan
  • Ran bin/storage upgrade, got a clean schema.
  • Reindexed all revisions with an external script.
  • Reviewed index via debug UI, saw appropriate index for non-repositoy revisions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Mar 15 2021, 10:37 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 15 2021, 11:16 PM
This revision was automatically updated to reflect the committed changes.