Page MenuHomePhabricator
Differential D9850

Move task/project storage to edges
ClosedPublic
Actions

Authored by epriestley on Jul 6 2014, 3:09 PM.
Tags
None
Referenced Files
F18371166: D9850.diff
Thu, Aug 28, 4:46 AM
F18111700: D9850.id.diff
Tue, Aug 12, 11:18 AM
F18099594: D9850.id23643.diff
Sat, Aug 9, 6:39 AM
F18093138: D9850.id23641.diff
Thu, Aug 7, 3:54 PM
F18090486: D9850.id23621.diff
Wed, Aug 6, 4:03 PM
F18068134: D9850.id.diff
Mon, Aug 4, 3:06 PM
F18067969: D9850.id23641.diff
Mon, Aug 4, 3:03 PM
F18067967: D9850.id23932.diff
Mon, Aug 4, 3:03 PM
View All Files
Subscribers
epriestley

Details

Reviewers
joshuaspence
btrahan
chad
Maniphest Tasks
T5245: Migrate Maniphest Projects to use edge infrastructure
Commits
Restricted Diffusion Commit
rPaa7953978975: Move task/project storage to edges
Summary

Ref T5245. This moves the actual storage over and stops reads and writes to the old table.

Test Plan
  • Verified tasks retained projects across the migration.
  • Added and removed projects from tasks.
  • Searched for: all, any, users' projects, not-in-projects, no-projects.

Diff Detail

Repository
rP Phabricator
Branch
projedge9
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1573
Build 1574: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 23621.Jul 6 2014, 3:09 PM
epriestley retitled this revision from to Move task/project storage to edges.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, chad, btrahan.
epriestley added a task: T5245: Migrate Maniphest Projects to use edge infrastructure.
Herald added a subscriber: epriestley. · View Herald TranscriptJul 6 2014, 3:09 PM
Harbormaster completed remote builds in B1553: Diff 23621.Jul 6 2014, 3:11 PM
joshuaspence added a parent revision: D9849: Modernize Project/Object edges.Jul 6 2014, 10:21 PM
joshuaspence accepted this revision.Jul 7 2014, 12:19 AM
joshuaspence edited edge metadata.
joshuaspence added inline comments.
resources/sql/autopatches/20140706.pedge.1.sql
7

I guess that we can drop this table? (probably later though)

This revision is now accepted and ready to land.Jul 7 2014, 12:19 AM
epriestley added a comment.Jul 7 2014, 1:03 AM

Yeah, I'll nuke it at the end of the chain. I renamed it locally and haven't hit any surprises yet.

epriestley updated this revision to Diff 23641.Jul 7 2014, 5:16 PM
epriestley edited edge metadata.
  • Rebasing through...
Harbormaster completed remote builds in B1573: Diff 23641.Jul 7 2014, 5:18 PM
epriestley updated this revision to Diff 23643.Jul 7 2014, 5:21 PM
  • Modern edgetype class names.
Harbormaster completed remote builds in B1575: Diff 23643.Jul 7 2014, 5:22 PM
epriestley closed this revision.Jul 17 2014, 10:42 PM
epriestley updated this revision to Diff 23932.

Closed by commit rPaa7953978975 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
10 lines
src/
applications/
maniphest/
controller/
5 lines
query/
52 lines
storage/
41 lines

Diff 23641

View Options

resources/sql/autopatches/20140706.pedge.1.sql

Loading...
View Options

src/applications/maniphest/controller/ManiphestReportController.php

Loading...
View Options

src/applications/maniphest/query/ManiphestTaskQuery.php

Loading...
View Options

src/applications/maniphest/storage/ManiphestTaskProject.php

Loading...
Phabricator · Built by Phacility