Skip to content

Commit c28ed2c

Browse files
authored
chore(commits): Remove dual_write_commit helper (#102502)
We're no longer planning on using the dual written tables, so removing these functions <!-- Describe your PR here. -->
1 parent cbd70a4 commit c28ed2c

File tree

2 files changed

+0
-35
lines changed

2 files changed

+0
-35
lines changed

src/sentry/releases/commits.py

Lines changed: 0 additions & 31 deletions
This file was deleted.

tests/sentry/tasks/test_commit_context.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
PullRequestCommit,
3434
)
3535
from sentry.models.repository import Repository
36-
from sentry.releases.commits import _dual_write_commit
3736
from sentry.shared_integrations.exceptions import ApiError
3837
from sentry.silo.base import SiloMode
3938
from sentry.tasks.commit_context import PR_COMMENT_WINDOW, process_commit_context
@@ -220,7 +219,6 @@ def test_success_existing_commit(
220219
author=self.commit_author,
221220
key="existing-commit",
222221
)
223-
_dual_write_commit(existing_commit)
224222
existing_commit.update(message="")
225223
assert Commit.objects.count() == 2
226224
event_frames = get_frame_paths(self.event)
@@ -288,7 +286,6 @@ def test_success_updating_group_owner(self, mock_get_commit_context, mock_record
288286
key="existing-commit",
289287
)
290288
existing_commit.update(message="")
291-
_dual_write_commit(existing_commit)
292289
assert Commit.objects.count() == 2
293290
event_frames = get_frame_paths(self.event)
294291
process_commit_context(
@@ -746,7 +743,6 @@ def test_time_threshold_filtering(
746743
key="existing-commit",
747744
)
748745
existing_commit.update(message="")
749-
_dual_write_commit(existing_commit)
750746

751747
# Map blame names to actual blame objects
752748
blame_mapping = {

0 commit comments

Comments
 (0)