Commit c4701ec7 authored by Kati Paizee's avatar Kati Paizee

Merge branch 'jejacks0n/add-clarification-for-saas-analytics-columns' into 'master'

Add clarity for analytics only database columns

See merge request gitlab-org/gitlab!67634
parents 4ca6ae85 e6eeef6d
......@@ -1145,3 +1145,7 @@ Note that the metrics linked here are GitLab-internal only:
- [Size](https://thanos.gitlab.net/graph?g0.range_input=2h&g0.max_source_resolution=0s&g0.expr=topk(500%2C%20max%20by%20(relname)%20(pg_total_relation_size_bytes%7Benvironment%3D%22gprd%22%7D))&g0.tab=1) is greater than 10 GB
Any table which has some high read operation compared to current [high-traffic tables](https://gitlab.com/gitlab-org/gitlab/-/blob/master/rubocop/rubocop-migrations.yml#L4) might be a good candidate.
As a general rule, we discourage adding columns to high-traffic tables that are purely for
analytics or reporting of GitLab.com. This can have negative performance impacts for all
self-managed instances without providing direct feature value to them.
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