Page MenuHomePhabricator
Differential D15216

Remove three ancient columns from Maniphest tasks: attached, projectPHIDs, ccPHIDs
ClosedPublic
Actions

Authored by epriestley on Feb 8 2016, 8:24 PM.
Tags
None
Referenced Files
F15467279: D15216.id36730.diff
Thu, Apr 3, 1:56 PM
F15454490: D15216.diff
Sat, Mar 29, 6:45 PM
F15449121: D15216.id36730.diff
Fri, Mar 28, 8:28 AM
F15445095: D15216.id36730.diff
Thu, Mar 27, 12:26 PM
F15438585: D15216.id36730.diff
Wed, Mar 26, 1:46 AM
F15422848: D15216.id36730.diff
Sat, Mar 22, 11:04 AM
F15399369: D15216.id36730.diff
Mon, Mar 17, 4:13 AM
F15385316: D15216.diff
Fri, Mar 14, 10:14 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP32225d1dd0f8: Remove three ancient columns from Maniphest tasks: attached, projectPHIDs…
Summary

Before edges, we stored some of this stuff directly on tasks.

  • attached was migrated to edges in Jan 2013.
  • projectPHIDs was never used, as far as I can tell?
  • ccPHIDs was migrated away and dropped more than a year ago.

None of these columns are used in modern code (instead, modern code uses edges).

Test Plan

grep, browsed around, bin/storage upgrade, unit tests.

Diff Detail

Repository
rP Phabricator
Branch
board6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10570
Build 12955: Run Core Tests
Build 12954: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36730.Feb 8 2016, 8:24 PM
epriestley retitled this revision from to Remove three ancient columns from Maniphest tasks: attached, projectPHIDs, ccPHIDs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.Feb 8 2016, 8:38 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 8 2016, 8:38 PM
Closed by commit rP32225d1dd0f8: Remove three ancient columns from Maniphest tasks: attached, projectPHIDs… (authored by epriestley, committed by epriestley). · Explain WhyFeb 8 2016, 10:10 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in D15315: Redesign PonderQuestionView.Feb 23 2016, 2:01 PM

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
patches/
28 lines
27 lines
src/
applications/
maniphest/
storage/
17 lines

Diff 36730

View Options

resources/sql/autopatches/20160208.task.1.sql

Loading...
View Options

resources/sql/autopatches/20160208.task.2.sql

Loading...
View Options

resources/sql/patches/migrate-maniphest-dependencies.php

Loading...
View Options

resources/sql/patches/migrate-maniphest-revisions.php

Loading...
View Options

src/applications/maniphest/storage/ManiphestTask.php

Loading...
Phabricator · Built by Phacility