Code Review: Add sharding keys to all your cell local tables
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
Important
Marking all tables that were blocked by the notes backfill with this message. Please read through #444222 (comment 2765183660) as it might affect the structure of the sharding key for this table
Overview
Related issue: Discover Cells 1.0 impact for code_review (#434975 - closed)
Add sharding keys to all your cell local tables, see the documentation and progress dashboard.
- All tables with the following gitlab_schema are considered “cell-local”:
gitlab_main_cellgitlab_ci
Tables
| Table | Schema | Sharding Key |
|---|---|---|
| approval_merge_request_rules_approved_approvers | gitlab_main_cell |
project_id (desired) |
| approver_groups | gitlab_main_cell |
group_id |
| approvers | gitlab_main_cell |
#444234 (closed) |
| commit_user_mentions | #550692 | |
| diff_note_positions | #550693 | |
| draft_notes | gitlab_main_cell |
project_id (desired) |
gitlab_main_cell |
Shared table, assigned to another group: #462801 (closed) | |
| merge_request_assignees | gitlab_main_cell |
project_id (desired) |
| merge_request_cleanup_schedules | gitlab_main_cell |
project_id (desired) |
| merge_request_context_commit_diff_files | gitlab_main_cell |
#444232 (closed) |
| merge_request_context_commits | gitlab_main_cell |
#444232 (closed) |
| merge_request_diff_commit_users |
gitlab_main_cell |
#472044 (closed) #502766 (closed) |
| merge_request_diff_commits | gitlab_main_cell |
project_id (desired) |
| merge_request_diff_files | gitlab_main_cell |
project_id (desired) |
| merge_request_diff_llm_summaries | gitlab_main_cell |
project_id (desired) |
| merge_request_diffs | gitlab_main_cell |
project_id |
| merge_request_metrics | gitlab_main_cell |
{"target_project_id"=>"projects"} |
| merge_request_predictions | gitlab_main_cell |
project_id (desired) |
| merge_request_review_llm_summaries | gitlab_main_cell |
project_id (desired) |
| merge_request_reviewers | gitlab_main_cell |
project_id (desired) |
| merge_requests | gitlab_main_cell |
{"target_project_id"=>"projects"} |
| merge_requests_closing_issues | gitlab_main_cell |
project_id (desired) |
| note_diff_files | gitlab_main_cell |
#550694 |
| note_metadata | gitlab_main_cell |
#550695 |
| notes | gitlab_main_cell |
#444222 (closed) |
| reviews | gitlab_main_cell |
project_id |
| suggestions | gitlab_main_cell |
#550696 |
| target_branch_rules | gitlab_main_cell |
project_id |
Edited by Mario Celi