diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-03-22 15:09:49 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-03-22 15:09:49 +0000 |
commit | f2dfd9ee819afb07bf11bd36a5f9d23009be0d39 (patch) | |
tree | edd9468dc9c6c55f9882175fd83a1aadec22edf0 /doc/development/database_review.md | |
parent | 058c34839488502fcec48d805b83728f928a318c (diff) | |
download | gitlab-ce-f2dfd9ee819afb07bf11bd36a5f9d23009be0d39.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'doc/development/database_review.md')
-rw-r--r-- | doc/development/database_review.md | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/doc/development/database_review.md b/doc/development/database_review.md index 0cd7e083e2a..9c428c62434 100644 --- a/doc/development/database_review.md +++ b/doc/development/database_review.md @@ -81,7 +81,7 @@ the following preparations into account. #### Preparation when adding migrations -- Ensure `db/schema.rb` is updated. +- Ensure `db/structure.sql` is updated. - Make migrations reversible by using the `change` method or include a `down` method when using `up`. - Include either a rollback procedure or describe how to rollback changes. - Add the output of the migration(s) to the MR description. @@ -137,7 +137,7 @@ the following preparations into account. - [Check indexes are present for foreign keys](migration_style_guide.md#adding-foreign-key-constraints) - Ensure that migrations execute in a transaction or only contain concurrent index/foreign key helpers (with transactions disabled) - - Check consistency with `db/schema.rb` and that migrations are [reversible](migration_style_guide.md#reversibility) + - Check consistency with `db/structure.sql` and that migrations are [reversible](migration_style_guide.md#reversibility) - Check queries timing (If any): Queries executed in a migration need to fit comfortably within `15s` - preferably much less than that - on GitLab.com. - For column removals, make sure the column has been [ignored in a previous release](what_requires_downtime.md#dropping-columns) |