Skip to content

Drop EventAttachment.file_id, part 1 #93806

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Swatinem
Copy link
Member

First part of dropping the column, marking it as pending.

@Swatinem Swatinem requested a review from a team June 18, 2025 13:37
@Swatinem Swatinem self-assigned this Jun 18, 2025
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 18, 2025
Copy link

codecov bot commented Jun 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #93806       +/-   ##
===========================================
+ Coverage   46.06%   88.02%   +41.95%     
===========================================
  Files       10323    10339       +16     
  Lines      595898   596635      +737     
  Branches    23171    23171               
===========================================
+ Hits       274521   525202   +250681     
+ Misses     320918    70974   -249944     
  Partials      459      459               

@Swatinem Swatinem force-pushed the swatinem/rm-eventattachment-fileid-part1 branch from 87208ad to 1aa35a4 Compare June 18, 2025 13:55
@Swatinem Swatinem force-pushed the swatinem/rm-eventattachment-fileid-part2 branch from 2efed5e to fa3be1a Compare June 18, 2025 13:59
Copy link
Contributor

This PR has a migration; here is the generated SQL for src/sentry/migrations/0930_rm_eventattachment_fileid_part1.py

for 0930_rm_eventattachment_fileid_part1 in sentry

--
-- Moved eventattachment.file_id field to pending deletion state
--
-- (no-op)

@Swatinem Swatinem force-pushed the swatinem/rm-eventattachment-fileid-part1 branch 2 times, most recently from 86da06d to 27ebb34 Compare June 18, 2025 15:15
Base automatically changed from swatinem/rm-eventattachment-fileid-part1 to master June 18, 2025 16:08
@Swatinem Swatinem force-pushed the swatinem/rm-eventattachment-fileid-part2 branch from fa3be1a to 1630998 Compare June 18, 2025 16:12
Copy link
Contributor

This PR has a migration; here is the generated SQL for src/sentry/migrations/0931_rm_eventattachment_fileid_part1.py

for 0931_rm_eventattachment_fileid_part1 in sentry

--
-- Moved eventattachment.file_id field to pending deletion state
--
-- (no-op)

@Swatinem Swatinem marked this pull request as ready for review June 18, 2025 16:47
@Swatinem Swatinem requested a review from a team as a code owner June 18, 2025 16:48
First part of dropping the column, marking it as pending.
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.

4 participants