Fixes T9117. Adds a migration to remove ponder vote data.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T9117: Remove unused edges after Ponder comes out of beta
- Commits
- rP9a52492f1b2c: Added autopatch to remove ponder vote data
I added a bunch of lines to phabricator_user.edge with type 18 and they were successfully removed by this patch
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- DeletePonderEdges (branched from master)
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 13451 Build 17288: Run Core Tests Build 17287: arc lint + arc unit
Event Timeline
Comment Actions
Task ref? You should probably find the mechanism by which these are written and read, and test that rather than looking at SQL?
Comment Actions
Task ref? You should probably find the mechanism by which these are written and read, and test that rather than looking at SQL?
@yelirekim if I understand correctly that mechanism doesn't exist anymore since D13833
Comment Actions
Yeah, this is just cleaning up free-floating garbage that we were hesitant to destroy when we unlinked it from everything.
(I actually don't have any of these locally myself, either.)
One minor double-check that I didn't mention is looking at Config → Edge Types and verifying that these constants don't appear there (they don't), just as a sanity check that the numbers aren't wildly wrong.