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
F15462185: D9850.id.diff
Tue, Apr 1, 12:54 PM
F15462184: D9850.diff
Tue, Apr 1, 12:53 PM
F15448212: D9850.id23643.diff
Fri, Mar 28, 3:28 AM
F15446696: D9850.id23641.diff
Thu, Mar 27, 7:50 PM
F15430505: D9850.id.diff
Mon, Mar 24, 7:59 AM
F15430074: D9850.id23621.diff
Mon, Mar 24, 5:34 AM
F15428789: D9850.id.diff
Sun, Mar 23, 10:19 PM
F15427967: D9850.id23621.diff
Sun, Mar 23, 5:26 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 1575
Build 1576: [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 23643

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