Database Reviews
I keep track of pending database reviews as a list of MRs and check them off as I go. As discussed with @Finotto today, the idea is to create one issue "Database Reviews" per milestone and maintain a list of pending database reviews there. /cc @dawsmith
The issue will be assigned to @NikolayS and @abrandl . The idea is to react to pings on MRs and link the MR in this list. The goal is to distribute review work between the two of us and also communicate the amount of reviews we're doing.
I tend to put duplicates on the list if I'm getting pinged repeatedly. Every ping-cycle means a distraction and for me it was worth tracking this.
This is my current list - let's use this issue to extend the list for this milestone and create a new issue next milestone:
-
~Community https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21719#note_104392420 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21521#note_104568347 -
~"Unblocking others" Kamil https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7582 -
~"Unblocking others" https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21893#note_105140458 -
~"Unblocking others" https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21767#note_105406114 -
~Urgent ~"Unblocking others" https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2528#note_135915 https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2528#note_136247 https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2528#note_136251 -
gitlab-restore https://gitlab.com/gitlab-restore/postgres-gprd/merge_requests/19#note_105154860 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21893#note_105824379 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22055#note_105882003 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7738 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7386#note_105203938 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22014 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7433/diffs -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7493 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7779 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21893 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7578#note_106070226 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7757#note_106315671 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22080#note_106119664 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21767#note_106084358 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22041#note_106203774 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22008#note_106238381 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22013#note_106303012 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7341#note_106332416 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/4213#note_106489662 -
https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2528#note_136247 -
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22143#note_106796618 -
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6947