Page MenuHomePhabricator

Remove unused edges after Ponder comes out of beta
Closed, ResolvedPublic

Description

See D13833#150619

Wherever we up after Unbeta Ponder, we should clean up the data.

Revisions and Commits

Restricted Differential Revision

Event Timeline

chad raised the priority of this task from to Normal.
chad updated the task description. (Show Details)
chad added a project: Ponder.
chad added subscribers: chad, epriestley.
chad removed chad as the assignee of this task.Nov 14 2015, 3:41 AM

This is similar to T9982. Complete that task first, which describes the SQL migration process in more detail.

This task covers more data (described in D13833#150619) which we can delete (we haven't written or read from it in more than a year). That comment accurately describes the patch that the migration should apply.

This will be somewhat difficult to convincingly test locally, since there hasn't been any way to vote on Ponder answers for more than a year. You could try putting some fake data in that table and make sure it gets deleted, and/or I can just test that it really works by running it locally since I probably still have some.

jcox added a revision: Restricted Differential Revision.Aug 25 2016, 6:57 PM
jcox closed this task as Resolved by committing Restricted Diffusion Commit.Aug 25 2016, 8:01 PM
jcox added a commit: Restricted Diffusion Commit.