-
Notifications
You must be signed in to change notification settings - Fork 8
[TILES-V2-2]: add new column to table_metadata #973
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
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 772 Passed, 0 Skipped, 2m 26s Total Time |
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, tested and confirmed db
column is created and defaults to ddb
during migration
Merge activity
|
1dacf89
to
935c534
Compare
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.
✅ BugBot reviewed your changes and found no bugs!
BugBot free trial expires on June 9, 2025
You have used $0.00 of your $1.00 spend limit so far. Manage your spend limit in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
Changes
Added a new
db
column to thetable_metadata
table to distinguish between PostgreSQL and DynamoDB tables.What changed?
20250506071427_add_table_metadata_db_column.ts
that adds adb
column to thetable_metadata
table with a default value of 'ddb' and not nullable constraintTableMetadata
model to include the newdb
property with type 'pg' | 'ddb'How to test?