Page MenuHomePhabricator
Create Task
Maniphest T12104

Clean up remnants of old Differential draft storage
Closed, ResolvedPublic
Actions

Description

The DifferentialDraft table no longer has readers or writers, but we should let it settle for a bit before getting rid of it.

Revisions and Commits

rP Phabricator
D18034rP04fd93e51e03 Drop DifferentialDraft storage

Related Objects

Event Timeline

epriestley created this task.Jan 13 2017, 3:45 PM
Herald added a subscriber: eadler. · View Herald TranscriptJan 13 2017, 3:45 PM
epriestley mentioned this in T12525: amckinley's Onboarding.Apr 9 2017, 1:46 PM
amckinley claimed this task.May 26 2017, 8:25 PM
amckinley added a subscriber: amckinley.

This looks pretty easy. No mentions of DifferentialDraft in the codebase other than the class itself and a single migration. Steps here (I think) are:

  • make the existing migration an empty file with a comment pointing here
  • remove the class
  • add a new migration to drop the existing table
epriestley added a comment.May 26 2017, 8:39 PM

That sounds right to me.

amckinley added a revision: D18034: Drop DifferentialDraft storage.May 26 2017, 8:51 PM
amckinley closed this task as Resolved by committing rP04fd93e51e03: Drop DifferentialDraft storage.May 26 2017, 8:59 PM
amckinley added a commit: rP04fd93e51e03: Drop DifferentialDraft storage.
Phabricator · Built by Phacility