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

Restricted Differential RevisionRestricted Diffusion Commit

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: Restricted Differential Revision.May 26 2017, 8:51 PM
amckinley closed this task as Resolved by committing Restricted Diffusion Commit.May 26 2017, 8:59 PM
amckinley added a commit: Restricted Diffusion Commit.
Phabricator · Built by Phacility