Some checks failed
Server CI / Postgres (shard 0) (push) Has been skipped
Server CI / Postgres (shard 3) (push) Has been skipped
BuildEnv Docker Image / build-image (push) Failing after 4s
BuildEnv Docker Image / build-image-fips (push) Failing after 4s
Migration Review & Release Notes / Generate migration review & release notes (push) Failing after 3s
Server CI / Build Environment (push) Failing after 3s
Server CI / Postgres (shard 1) (push) Has been skipped
Tools CI / check-style (mattermost-govet) (push) Failing after 4s
Server CI / Postgres (shard 2) (push) Has been skipped
Tools CI / check-style (mmgotool) (push) Failing after 3s
Server CI / Elasticsearch v8 Compatibility (push) Has been skipped
Server CI / OpenSearch v2 Compatibility (push) Has been skipped
Server CI / Postgres FIPS (shard 0) (push) Has been skipped
Tools CI / Test (mattermost-govet) (push) Failing after 3s
Server CI / Postgres FIPS (shard 1) (push) Has been skipped
Server CI / Postgres FIPS (shard 2) (push) Has been skipped
Tools CI / Test (mmgotool) (push) Failing after 3s
Server CI / Postgres FIPS (shard 3) (push) Has been skipped
Server CI / Run mmctl tests (push) Has been skipped
Server CI / Run mmctl tests (FIPS) (push) Has been skipped
Server CI / Merge Postgres Test Results (push) Has been skipped
Server CI / Merge Postgres FIPS Test Results (push) Has been skipped
Server CI / Check backport migrations (push) Has been skipped
Server CI / Check generated files (push) Has been skipped
Server CI / check-style (push) Has been skipped
Server CI / Vet API (push) Has been skipped
Server CI / Build mattermost server app (push) Has been skipped
Server CI / Server CI Complete (push) Failing after 2s
Server CI / Server CI Artifacts (push) Has been skipped
Server CI / Sentry Upload (push) Has been skipped
Server CI / Server CI Report (push) Failing after 5m1s
Server CI Nightly Race / Race Detector (push) Has been skipped
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (javascript) (push) Has been cancelled
Webapp CI Cache Warm / Warm Webapp node_modules Cache (push) Has been cancelled
E2E CI Cache Warm / Warm E2E npm Registry Cache (push) Has been cancelled
API / build (push) Has been cancelled
Web App CI / check-types (push) Has been cancelled
Web App CI / check-lint (push) Has been cancelled
Web App CI / check-i18n (push) Has been cancelled
Web App CI / check-external-links (push) Has been cancelled
Web App CI / test (platform) (push) Has been cancelled
Web App CI / test (mattermost-redux) (push) Has been cancelled
Web App CI / test (channels shard 1/4) (push) Has been cancelled
Web App CI / test (channels shard 2/4) (push) Has been cancelled
Web App CI / test (channels shard 3/4) (push) Has been cancelled
Web App CI / test (channels shard 4/4) (push) Has been cancelled
Web App CI / upload-coverage (push) Has been cancelled
Web App CI / build (push) Has been cancelled
YAML Lint / yamllint (push) Has been cancelled
Push mirrored docker images / cd/Push mirrored docker images (push) Has been cancelled
docs-cd / Build and deploy docs (push) Has been cancelled
Docs CI / build (push) Has been cancelled
E2E Cache Warm: Cypress deps / Warm Cypress deps cache (push) Has been cancelled
Scorecards supply-chain security / Scorecard analysis (push) Has been cancelled
Server CI Weekly / Postgres FIPS (push) Has been skipped
Server CI Weekly / Run mmctl tests (FIPS) (push) Has been skipped
Server CI Weekly / Postgres with binary parameters (push) Has been skipped
Server CI Weekly / Build Environment (push) Failing after 1s
Server CI Cache Warm / Warm Go Module and Build Cache (push) Failing after 2s
Server CI Nightly Race / Build Environment (push) Failing after 1s
62 lines
2.7 KiB
Markdown
62 lines
2.7 KiB
Markdown
# Database Migration Rules
|
|
|
|
Enforce these rules to prevent table locks and performance degradation on large production databases (100M+ posts).
|
|
|
|
## General Rules
|
|
|
|
- Each migration should do one thing. Don't combine unrelated schema changes.
|
|
- Prefer additive changes (new columns, new tables) over destructive ones.
|
|
- Schema migrations must **ALWAYS** be backwards compatible until the last ESR.
|
|
- Test migrations against a database with realistic data volume, not just a dev-sized dataset. An `EXPLAIN ANALYZE` that looks fine on 12M posts may behave very differently at 100M posts.
|
|
- If a shipped migration is broken, do not modify it. Add a new migration to correct.
|
|
|
|
## File Naming and Structure
|
|
|
|
Migration files follow the convention `{sequence_number}_{description}.{up|down}.sql` and live in `db/migrations/{driver_name}/`. After creating migration files, run `make migrations-extract` to update `db/migrations/migrations.list`. Merge upstream before submitting a PR to avoid sequence number collisions.
|
|
|
|
## Table Locking
|
|
|
|
Never write a migration that locks an entire table. Common operations that acquire table-level locks:
|
|
|
|
- `ALTER COLUMN` (type change) — rewrites the table
|
|
- `CREATE INDEX` without `CONCURRENTLY`
|
|
- `ADD FOREIGN KEY` without `NOT VALID`
|
|
- `LOCK TABLE`
|
|
|
|
## Column Type Changes
|
|
|
|
As noted above, `ALTER COLUMN` (type change) rewrites the entire table and blocks concurrent DML: strongly avoid. A real-world example took 8+ hours on a large customer database. When unavoidable, use a multi-release phased approach:
|
|
|
|
1. Release N: add the new column.
|
|
2. Release N+1: backfill data via background jobs; use triggers for ongoing updates.
|
|
3. Release N+2 (ESR): start using the new column in code.
|
|
4. Release N+3 (ESR): drop the old column.
|
|
|
|
## Batch Updates
|
|
|
|
Never do a full-table `UPDATE` in a migration. Process data in batches in a job at runtime:
|
|
|
|
```sql
|
|
UPDATE foo SET col = value
|
|
WHERE id IN (
|
|
SELECT id FROM foo WHERE id > :offset ORDER BY id ASC LIMIT 100
|
|
);
|
|
```
|
|
|
|
Store the offset and resume across job runs.
|
|
|
|
## Adding Unique Constraints
|
|
|
|
Do not add a unique constraint directly. Create the index concurrently first, then attach it:
|
|
|
|
```sql
|
|
CREATE UNIQUE INDEX CONCURRENTLY constraint_name ON foo (bar);
|
|
ALTER TABLE foo ADD UNIQUE USING INDEX constraint_name;
|
|
```
|
|
|
|
Note that this cannot be done inside a transaction block, so it must be in a separate migration file. Use the `-- morph:nontransactional` comment to disable transactions for that migration.
|
|
|
|
## Foreign Key Constraints
|
|
|
|
Avoid foreign key constraints. Adding a foreign key takes a `SHARE ROW EXCLUSIVE` lock, limiting concurrent activity to SELECT queries only. If truly needed, consider adding the constraint with `NOT VALID` to avoid the full-table scan under lock.
|