diff options
| author | Andreas Brandl <abrandl@gitlab.com> | 2019-06-21 13:16:05 +0000 |
|---|---|---|
| committer | Andreas Brandl <abrandl@gitlab.com> | 2019-06-21 13:16:05 +0000 |
| commit | 9c1ab68a58a66fde3b44c7a1af4d9ec02428ea9c (patch) | |
| tree | d28a45c910de09773b28c9ec049fcd933d7e2245 | |
| parent | 0889d0b93c1ff363086456972ae1a6f490f6176a (diff) | |
| download | gitlab-ce-ab-codereview-db.tar.gz | |
Clarify "expensive query"ab-codereview-db
| -rw-r--r-- | doc/development/code_review.md | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/doc/development/code_review.md b/doc/development/code_review.md index f795f17eb52..6123f9f845a 100644 --- a/doc/development/code_review.md +++ b/doc/development/code_review.md @@ -60,7 +60,7 @@ from teams other than your own. 1. If your merge request includes backend changes [^1], it must be **approved by a [backend maintainer](https://about.gitlab.com/handbook/engineering/projects/#gitlab-ce_maintainers_backend)**. - 1. If your merge request includes database migrations or changes to expensive queries [^1], it must be + 1. If your merge request includes database migrations or changes to expensive queries [^2], it must be **approved by a [database maintainer](https://about.gitlab.com/handbook/engineering/projects/#gitlab-ce_maintainers_database)**. 1. If your merge request includes frontend changes [^1], it must be **approved by a [frontend maintainer](https://about.gitlab.com/handbook/engineering/projects/#gitlab-ce_maintainers_frontend)**. @@ -379,3 +379,4 @@ Largely based on the [thoughtbot code review guide]. [team]: https://about.gitlab.com/team/ [build handbook]: https://about.gitlab.com/handbook/build/handbook/build#how-to-work-with-build [^1]: Please note that specs other than JavaScript specs are considered backend code. +[^2]: We encourage you to seek guidance from a database maintainer if your merge request is potentially introducing expensive queries. It is most efficient to comment on the line of code in question with the SQL queries so they can give their advice. |