Page MenuHomePhabricator

Destroy surplus columns
ClosedPublic

Authored by epriestley on Sep 27 2014, 11:12 PM.
Tags
None
Referenced Files
F15541198: D10591.id25465.diff
Fri, Apr 25, 11:19 AM
F15525492: D10591.id25465.diff
Mon, Apr 21, 2:49 PM
F15514397: D10591.id25427.diff
Fri, Apr 18, 2:23 AM
F15513003: D10591.id.diff
Thu, Apr 17, 3:33 PM
F15509969: D10591.diff
Wed, Apr 16, 5:44 PM
F15507072: D10591.diff
Tue, Apr 15, 4:07 PM
F15472384: D10591.id25427.diff
Sat, Apr 5, 12:55 PM
F15458979: D10591.id25465.diff
Mon, Mar 31, 8:53 AM

Details

Summary

Ref T1191. This destroys surplus columns:

  • Pholio's transaction comments have a mockID column, but this is not used. The imageID column is used instead.
  • Phragment has an unused description column.
  • Releeph has an unused summary column.
Test Plan
  • Grepped for usage of these columns.
  • Checked that these exist in production, too.
  • Ran upgrades.
  • Added Pholio inline comments.
  • Saw fewer warnings.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Destroy surplus columns.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Sep 30 2014, 4:50 PM
This revision was automatically updated to reflect the committed changes.