Skip to content
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

Define #build_change_column_definition for MySQL and PG #45751

Conversation

adrianna-chang-shopify
Copy link
Contributor

Summary

Define APIs on the MySQL and PostgreSQL connection adapters for building
ChangeColumnDefaultDefinition objects. These provide information on what a
column default change would look like, when called with the same arguments as
would be passed to #change_column_default.

Other Information

As with #change_column, we ignore SQLite because it recreates the table from scratch when performing change_column operations.

Define APIs on the MySQL and PostgreSQL connection adapters for building
ChangeColumnDefaultDefinition objects. These provide information on what a
column default change would look like, when called with the same arguments as
would be passed to #change_column_default.
@adrianna-chang-shopify adrianna-chang-shopify force-pushed the ac-build-change-column-default-definition branch from bd87098 to 7455336 Compare August 3, 2022 20:27
@eileencodes eileencodes merged commit 0c9e406 into rails:main Aug 4, 2022
@eileencodes
Copy link
Member

eileencodes commented Aug 4, 2022

That build was really flaky but none of the failures were in active record so I merged. Thanks @adrianna-chang-shopify!

@adrianna-chang-shopify adrianna-chang-shopify deleted the ac-build-change-column-default-definition branch August 5, 2022 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants