-
Notifications
You must be signed in to change notification settings - Fork 314
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
chore: remove deprecated merged schema field #3482
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3482 +/- ##
==========================================
+ Coverage 68.01% 68.03% +0.02%
==========================================
Files 330 330
Lines 53574 53560 -14
==========================================
+ Hits 36437 36440 +3
+ Misses 14740 14725 -15
+ Partials 2397 2395 -2
☔ View full report in Codecov by Sentry. |
ALTER TABLE wh_uploads | ||
DROP COLUMN IF EXISTS mergedschema; |
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.
Can we remove the column in a release/separate PR?
So we are able to revert back without manual intervention
Description
Getting rid of the deprecated
mergedschema
field from warehouse tablesNotion Ticket
Notion Link
Security