cloud: Premium supports data export#22648
cloud: Premium supports data export#22648shiyuhang0 wants to merge 7 commits intopingcap:release-8.5from
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces documentation for exporting data from TiDB Cloud Premium instances and configuring cross-account access for external storage providers like Amazon S3, Azure Blob Storage, and Alibaba Cloud OSS. The review feedback identifies several actionable improvements, including correcting list numbering errors, fixing incorrect navigation paths (e.g., changing "Import" to "Export"), and refining terminology to ensure clarity between source and target data in an export context. Additionally, the feedback addresses style guide violations such as passive voice usage, inconsistent capitalization, and unnecessary backticks.
There was a problem hiding this comment.
Pull request overview
Adds TiDB Cloud Premium documentation pages for exporting data to external object storage and for configuring the required external storage access, and wires them into the Premium TOC.
Changes:
- Add a new Premium “Export Data” doc with export locations, options, examples, cancel flow, speed, and pricing.
- Add a new Premium “Configure External Storage Access” doc for S3/Azure/OSS setup.
- Update
TOC-tidb-cloud-premium.mdto include the new Premium export docs.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.
| File | Description |
|---|---|
tidb-cloud/premium/premium-export.md |
New Premium export guide (locations/options/examples/cancel/speed/pricing). |
tidb-cloud/premium/external-storage.md |
New Premium guide for configuring external storage access (S3/Azure/OSS). |
TOC-tidb-cloud-premium.md |
Adds TOC entries pointing to the new Premium export docs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@xuanyu66: adding LGTM is restricted to approvers and reviewers in OWNERS files. DetailsIn response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
First-time contributors' checklist
What is changed, added or deleted? (Required)
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?