Adds a section about `change_column_default`
What does this MR do?
Includes a new section under our migration style guide for the usage of
change_column_default
. Particularly, specifying that it's not a risky
operation for larger tables and can be used in a single transaction
Related issues
Author's checklist
-
Follow the Documentation Guidelines and Style Guide. - [-] If applicable, update the permissions table.
-
Link docs to and from the higher-level index page, plus other related docs where helpful. -
Apply the documentation label.
Review checklist
All reviewers can help ensure accuracy, clarity, completeness, and adherence to the Documentation Guidelines and Style Guide.
1. Primary Reviewer
-
Review by a code reviewer or other selected colleague to confirm accuracy, clarity, and completeness. This can be skipped for minor fixes without substantive content changes.
2. Technical Writer
-
Optional: Technical writer review. If not requested for this MR, must be scheduled post-merge. To request for this MR, assign the writer listed for the applicable DevOps stage.
3. Maintainer
-
Review by assigned maintainer, who can always request/require the above reviews. Maintainer's review can occur before or after a technical writer review. -
Ensure a release milestone is set and that you merge the equivalent EE MR before the CE MR if both exist. -
If there has not been a technical writer review, create an issue for one using the Doc Review template.
Merge request reports
Activity
changed milestone to %12.4
added database docs-only documentation labels
added databasereview pending label
1 Warning ⚠ Please add a throughput label to this merge request. 2 Messages 📖 This merge request adds or changes files that require a review from the Database team. 📖 CHANGELOG missing: If this merge request doesn’t need a CHANGELOG entry, feel free to ignore this message. You can create one with:
bin/changelog -m 18090 "Adds a section about `change_column_default`"
If you want to create a changelog entry for GitLab EE, run the following instead:
bin/changelog --ee -m 18090 "Adds a section about `change_column_default`"
Note: Merge requests with ~backstage, ci-build, meta do not trigger this check.
This merge request requires a database review. To make sure these changes are reviewed, take the following steps:
- Ensure the merge request has database and databasereview pending labels. If the merge request modifies database files, Danger will do this for you.
- Use the Database changes checklist template or add the appropriate items to the MR description.
- Assign and mention the database reviewer suggested by Reviewer Roulette.
Generated by
🚫 DangerEdited by 🤖 GitLab Bot 🤖@abrandl could you please review? I'm trying to capture the past conversations on the DB hours.
PS. This is the last one I swear
🙈 Edited by Mayra Cabreraassigned to @abrandl and unassigned @mayra-cabrera
- Resolved by Achilleas Pipinellis
- Resolved by Mayra Cabrera
PS. This is the last one I swear
No, this is really great, thank you!
😄 @mayra-cabrera I left a comment as a potential addition, but feel free to merge as is, too.
Edited by Andreas Brandl
assigned to @mayra-cabrera and unassigned @abrandl
added databaseapproved label and removed databasereview pending label
added 104 commits
-
5a38a6ed...3630d71d - 103 commits from branch
master
- a5ef00dd - Add a section about change_column_default
-
5a38a6ed...3630d71d - 103 commits from branch
- Resolved by Achilleas Pipinellis
@axil could you please review?
assigned to @axil and unassigned @mayra-cabrera
mentioned in commit e5350659
This merge request has been deployed to the GitLab.com environment
gstg
in GitLab auto-deploy version12.4.201910130810-44adee04c32.30ae0ff6d6f
.A list of all the deployed commits can be found here.
🤖 If this message is incorrect, please create an issue in the Release Tools project.added workflowstaging label
This merge request has been deployed to the GitLab.com environment
gprd-cny
in GitLab auto-deploy version12.4.201910130810-44adee04c32.30ae0ff6d6f
.A list of all the deployed commits can be found here.
🤖 If this message is incorrect, please create an issue in the Release Tools project.added workflowcanary label and removed workflowstaging label
This merge request has been deployed to the GitLab.com environment
gprd
in GitLab auto-deploy version12.4.201910160011-08e6905db3d.4748e740ecf
.A list of all the deployed commits can be found here.
🤖 If this message is incorrect, please create an issue in the Release Tools project.added workflowproduction label and removed workflowcanary label
mentioned in commit Dalahro/gitlab@4d1a3d31
added typemaintenance label