Commit 57b7a404 authored by Russell Dickenson's avatar Russell Dickenson

Merge branch '220193-update-database-review-docs' into 'master'

Update database review docs to link to Telemetry guide

See merge request gitlab-org/gitlab!37050
parents 9752a3c8 1861b865
...@@ -19,6 +19,10 @@ A database review is required for: ...@@ -19,6 +19,10 @@ A database review is required for:
generally up to the author of a merge request to decide whether or generally up to the author of a merge request to decide whether or
not complex queries are being introduced and if they require a not complex queries are being introduced and if they require a
database review. database review.
- Changes in usage data metrics that use `count` and `distinct_count`.
These metrics could have complex queries over large tables.
See the [Telemetry Guide](telemetry/usage_ping.md#implementing-usage-ping)
for implementation details.
A database reviewer is expected to look out for obviously complex A database reviewer is expected to look out for obviously complex
queries in the change and review those closer. If the author does not queries in the change and review those closer. If the author does not
......
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