datastore: Revert use of materialized views (v14.4)
Revert the introduction of materialized views for valid_primaries
. As
it turns out, the changes cause incompatibilities with Postgres 11,
which is still actively in use. Furthermore, the performance issues we
have seen have not been fully fixed with this change, and we do not yet
fully understand the root cause for this.
Merge request reports
Activity
assigned to @pks-t
added devopscreate groupgitaly labels
1 Warning No milestone was set, nor could it be detected from the issues this merge request closes. Reviewer roulette
Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.
To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.
Once you've decided who will review this merge request, mention them as you normally would! Danger does not automatically notify them for you.
Category Reviewer Maintainer ~ James Fargher ( @proglottis
) (UTC+13, 12 hours ahead of@pks-t
)Christian Couder ( @chriscool
) (UTC+1, same timezone as@pks-t
)If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by GitalyBotadded regression sectiondev typebug labels
requested review from @samihiltunen and @8bitlife
mentioned in merge request !4104 (closed)
removed review request for @samihiltunen
Setting label(s) Category:Gitaly based on groupgitaly.
added Category:Gitaly label
mentioned in commit fe9eb6b4