-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Clean up ArtifactBundle(FlatFile)Index
, step 3
#66208
Conversation
This PR has a migration; here is the generated SQL for --
-- Raw SQL operation
--
DROP TABLE "sentry_artifactbundleflatfileindex";
--
-- Raw SQL operation
--
DROP TABLE "sentry_flatfileindexstate";
--
-- Raw SQL operation
--
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "release_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "dist_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "date_last_modified"; |
bce9612
to
6f57d2c
Compare
48ca11b
to
6079046
Compare
2afd2c3
to
164e231
Compare
6f57d2c
to
48c9124
Compare
This PR has a migration; here is the generated SQL for --
-- Raw SQL operation
--
DROP TABLE "sentry_artifactbundleflatfileindex";
--
-- Raw SQL operation
--
DROP TABLE "sentry_flatfileindexstate";
--
-- Raw SQL operation
--
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "release_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "dist_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "date_last_modified"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Schema changes look good.
operations = [ | ||
migrations.RunSQL( | ||
""" | ||
DROP TABLE "sentry_artifactbundleflatfileindex"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There aren't any references to this model or table in the application code anymore 👍
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "release_name"; | ||
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "dist_name"; | ||
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "date_last_modified"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These columns don't exist in the model definition currently 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, nice cleanup!
In step 3, we fully drop the obsolete columns and tables after they have been removed from the migration state in prior steps.
48c9124
to
c472247
Compare
This PR has a migration; here is the generated SQL for --
-- Raw SQL operation
--
DROP TABLE "sentry_artifactbundleflatfileindex";
--
-- Raw SQL operation
--
DROP TABLE "sentry_flatfileindexstate";
--
-- Raw SQL operation
--
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "release_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "dist_name";
ALTER TABLE "sentry_artifactbundleindex" DROP COLUMN "date_last_modified"; |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #66208 +/- ##
=======================================
Coverage 84.29% 84.29%
=======================================
Files 5309 5309
Lines 237222 237222
Branches 41038 41038
=======================================
Hits 199974 199974
Misses 37029 37029
Partials 219 219 |
This is the third step, following #66206 and #66207:
In step 3, we fully drop the obsolete columns and tables after they have been removed from the migration state in prior steps.