Commit 8f2bdeb7 authored by Yorick Peterse's avatar Yorick Peterse

Merge branch 'change_migration_style_guide' into 'master'

[Migration guide] Add a note about prefering change_column_null over change_column

See merge request !11473
parents 9e34c8dc cf9d9d86
...@@ -139,6 +139,8 @@ Adding or removing a NOT NULL clause (or another constraint) can typically be ...@@ -139,6 +139,8 @@ Adding or removing a NOT NULL clause (or another constraint) can typically be
done without requiring downtime. However, this does require that any application done without requiring downtime. However, this does require that any application
changes are deployed _first_. Thus, changing the constraints of a column should changes are deployed _first_. Thus, changing the constraints of a column should
happen in a post-deployment migration. happen in a post-deployment migration.
NOTE: Avoid using `change_column` as it produces inefficient query because it re-defines
the whole column type. For example, to add a NOT NULL constraint, prefer `change_column_null `
## Changing Column Types ## Changing Column Types
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment