Page MenuHomePhabricator

Rename the repository database
AbandonedPublic

Authored by joshuaspence on Jun 9 2015, 12:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 12:49 AM
Unknown Object (File)
Sat, Apr 6, 10:13 PM
Unknown Object (File)
Sat, Apr 6, 3:41 PM
Unknown Object (File)
Sat, Apr 6, 2:20 AM
Unknown Object (File)
Tue, Apr 2, 8:56 AM
Unknown Object (File)
Mar 20 2024, 6:11 PM
Unknown Object (File)
Mar 9 2024, 4:28 PM
Unknown Object (File)
Feb 8 2024, 5:59 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 6977
Build 7013: [Placeholder Plan] Wait for 30 Seconds
Build 7012: arc lint + arc unit

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.