Commit 1861b865 authored by Alina Mihaila's avatar Alina Mihaila Committed by Russell Dickenson

Update database review docs to link to Telemetry guide

parent 1ad2429e
...@@ -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