Page MenuHomePhabricator

Rename the repository database
AbandonedPublic

Authored by joshuaspence on Jun 9 2015, 12:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 6:02 PM
Unknown Object (File)
Thu, Nov 21, 8:39 PM
Unknown Object (File)
Mon, Nov 18, 5:15 AM
Unknown Object (File)
Thu, Nov 14, 11:37 AM
Unknown Object (File)
Mon, Nov 11, 12:40 PM
Unknown Object (File)
Sun, Nov 10, 1:36 PM
Unknown Object (File)
Sun, Nov 10, 12:28 PM
Unknown Object (File)
Wed, Nov 6, 3:27 AM

Details

Summary

Fixes T8476. Rename the phabricator_repository database to phabricator_diffusion for consistency.

Test Plan

This doesn't actually work, but gives an idea of how large this change is.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6927
Build 6949: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Rename the repository database.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

Still left to do:

  • Figure out what to do with the *CommitRef classes? We currently have DiffusionCommitRef and PhabricatorRepositoryCommitRef.
  • Fix the database migrations. Specifically, I suspect we need to use LiskRawMigrationIterator instead of LiskMigrationIterator.
joshuaspence edited edge metadata.

Fairly hacky way to make things work

Although it mostly works, I am abandoning this diff because keeping it up-to-date seems like more effort than I am willing to give. It's not clear to me at this stage when (if ever) T8476 is likely to be resolved.