Add details about some backward incompatible operations

- adding a NOT NULL column
- dropping a column
- sqlite special case
This commit is contained in:
David Wobrock committed 2021-12-20 22:03:28 +01:00
1 parent 967047d4e6
commit 1f8476fc43
1 file changed
+79
+79
View File
@@ -42,3 +42,82 @@ You can ignore checks through the `--exclude-migration-tests` option by specifyi
|`CREATE_INDEX` | (Postgresql specific) Creating an index without the concurrent keyword will lock the table and may generate downtime | Warning
|`DROP_INDEX` | (Postgresql specific) Dropping an index without the concurrent keyword will lock the table and may generate downtime | Warning
|`REINDEX` | (Postgresql specific) Reindexing will lock the table and may generate downtime | Warning
## Details about backward incompatibilities
This section will go into the depth of the different check the migration linter makes.
The base hypotheses of these cases are:
- in a production system, you cannot deploy your database(s) (DB) and code server(s) simultaneously
- you deploy your DB first, as there are very few cases in which deploying the code first is viable for database operations
### Adding `NOT NULL` column without default value
A frequent and error-prone operation is adding a non-nullable column to an existing table.
First, adding a `NOT NULL` column **without any default value** is problematic.
**Forward migration**:
1. update your DB to add a `NOT NULL` column
2. your Django code will not specify the new column when inserting a row
=> error `column cannot be null`
3. once the code updated, insertion will work where the new column has explicitly been specified
**Rollback**: in the case of a rollback, you will encounter the same errors.
**Incorrect solution**: specify a default value in the Django model field.
One would think that adding a default value in Django will prevent these errors.
A common misconception is that the Django default value is translated to a database default.
But Django actually uses the default value to fill existing rows and set an unspecified column value to its default.
The latter is done at the application level, by Django - and the database default value was dropped during migration.
You can read more about this in the [Django and its default values blog post](https://medium.com/botify-labs/django-and-its-default-values-c21a13cff9f).
**Forward migration**:
1. update your DB to add a `NOT NULL` column with a Django default
2. your Django code will not specify the new column when inserting a row, and Django is not aware of the default value
=> error `column cannot be null`
3. once the code updated, insertion will work
**Rollback**: in the case of a rollback, you will encounter the same errors
**Solutions**:
- Make the column nullable
- Set a database default using Django's [RunSQL](https://docs.djangoproject.com/en/dev/ref/migration-operations/#django.db.migrations.operations.RunSQL)
- Set a database default using [django-add-default-value](https://github.com/3YOURMIND/django-add-default-value/)
### Dropping a column
Deletion operations often lead to errors during deployment.
**Forward migration**:
1. update your DB to drop a column
2. your code will crash retrieving rows from this table (because Django explicits all column names when fetching a model object)
3. once the code is updated, the errors should cease
**Rollback**:
1. rollback your code
2. your code will crash retrieving rows from this table (because Django explicits all column names when fetching a model object)
3. rollback your DB to re-create the column
4. restore a backup of your data (if available and fresh enough)
**Solutions**:
- Deprecate the column before dropping it using [django-deprecate-fields](https://github.com/3YOURMIND/django-deprecate-fields/)
- Don't actually drop the column, but fake the drop migration until you are sure you won't roll back.
Be careful /!\ fake dropping a non-nullable column without a database default will create errors once the code is not aware of the column anymore.
### Others
To be added...
## The special case of sqlite
While on PostgreSQL and MySQL a table modification can be expressed by one `ALTER TABLE` statement, sqlite is handled in a different way.
For operations like adding a column to an existing table, Django actually generates four statements:
- creating a new table with the new schema
- copying all rows from the current table to the new one
- dropping the current table
- renaming the new table to the current table name
At the time of writing, the linter doesn't support a fine-grained detection of field alteration when using the sqlite process.
An [issue #142](https://github.com/3YOURMIND/django-migration-linter/issues/142) is already open and Django also has a [ticket about supporting sqlite ALTER functions](https://code.djangoproject.com/ticket/32502).