Skip to content

Updating database MR guide with requirements

Craig Gomes requested to merge cg-update-db-mr-reqs into master

What does this MR do?

This is a documentation update to indicate the required fields for a database review. We are now requiring query plans for all queries and the raw sql output (rather than recommending). The updated documentation also indicates that these missing files will require that reviewers and maintainers send back to the original author.

The goal here is to be more efficient with our db reviews and to better leverage our knowledge across all of the development team. The missing query plans and raw sql were common themes as areas of improvement from this recent issue - Gather Database Maintainer Feedback - Time spent, common themes, suggested improvements

Relates to gitlab-org/database-team/team-tasks#114 (closed)

Edited by Craig Gomes

Merge request reports

Loading