diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2018-07-24 18:28:42 +0000 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2018-07-24 18:28:42 +0000 |
| commit | f5b45519dba89296cbd6f350f5c29b655d5aef8d (patch) | |
| tree | 6bf07c51e165a7a1c04adcfcdd4511b1290bb57c | |
| parent | 39cb42c329148279f5acd21cf132e79d756e0195 (diff) | |
| parent | 378610372fae447b9af00ebecfca94a2edc3c4b7 (diff) | |
| download | gitlab-ce-f5b45519dba89296cbd6f350f5c29b655d5aef8d.tar.gz | |
Merge branch 'osw-background-migrations-best-practice-doc' into 'master'
Add best practice points regarding background migrations on huge tables
See merge request gitlab-org/gitlab-ce!20815
| -rw-r--r-- | doc/development/background_migrations.md | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/doc/development/background_migrations.md b/doc/development/background_migrations.md index 46c5baddb9c..16195cbbbdf 100644 --- a/doc/development/background_migrations.md +++ b/doc/development/background_migrations.md @@ -296,10 +296,18 @@ for more details. ## Best practices +1. Make sure to know how much data you're dealing with 1. Make sure that background migration jobs are idempotent. 1. Make sure that tests you write are not false positives. 1. Make sure that if the data being migrated is critical and cannot be lost, the clean-up migration also checks the final state of the data before completing. +1. Make sure to know how much time it'll take to run all scheduled migrations +1. When migrating many columns, make sure it won't generate too many + dead tuples in the process (you may need to directly query the number of dead tuples + and adjust the scheduling according to this piece of data) +1. Make sure to discuss the numbers with a database specialist, the migration may add + more pressure on DB than you expect (measure on staging, + or ask someone to measure on production) [migrations-readme]: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/spec/migrations/README.md [issue-rspec-hooks]: https://gitlab.com/gitlab-org/gitlab-ce/issues/35351 |