Skip to content

Conversation

@wedamija
Copy link
Member

Follow up migration to delete these from the database. Leaving this in draft until the code removal deploys

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 31, 2025
@wedamija wedamija force-pushed the danf/commit-remove-dual-write-tables branch from 2822b79 to 2642a0b Compare October 31, 2025 21:31
Base automatically changed from danf/commit-remove-dual-write-tables to master October 31, 2025 21:52
Follow up migration to delete these from the database.
@wedamija wedamija force-pushed the danf/commit-delete-dual-write-tables branch from f81980f to 55288c1 Compare November 3, 2025 17:39
@wedamija wedamija marked this pull request as ready for review November 3, 2025 17:39
@wedamija wedamija requested a review from a team as a code owner November 3, 2025 17:39
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

This PR has a migration; here is the generated SQL for src/sentry/releases/migrations/0003_real_delete_dual_written_commit_tables.py

for 0003_real_delete_dual_written_commit_tables in releases

--
-- Delete model Commit
--
DROP TABLE "releases_commit" CASCADE;
--
-- Delete model CommitFileChange
--
DROP TABLE "releases_commitfilechange" CASCADE;

@getsentry getsentry deleted a comment from codecov bot Nov 3, 2025
@getsentry getsentry deleted a comment from github-actions bot Nov 3, 2025
@getsentry getsentry deleted a comment from sentry bot Nov 3, 2025
@getsentry getsentry deleted a comment from cursor bot Nov 3, 2025

operations = [
SafeDeleteModel(name="Commit", deletion_action=DeletionAction.DELETE),
SafeDeleteModel(name="CommitFileChange", deletion_action=DeletionAction.DELETE),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wedamija Do you want DeletionAction.MOVE_TO_PENDING first?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already handled in 0002. Also, jfyi, ci will complain if someone attempts to delete before doing MOVE_TO_PENDING first

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to know!

@wedamija wedamija merged commit fa9c22b into master Nov 5, 2025
67 checks passed
@wedamija wedamija deleted the danf/commit-delete-dual-write-tables branch November 5, 2025 18:54
priscilawebdev pushed a commit that referenced this pull request Nov 6, 2025
Follow up migration to delete these from the database.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants