Skip to content

test0412-2 #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

test0412-2 #36

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
be56261
Update br-checkpoint-backup.md
qiancai Apr 21, 2025
7b725f1
Update br-checkpoint-backup.md
qiancai Apr 21, 2025
57313fc
Update br-checkpoint-backup.md
qiancai Apr 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions br/br-checkpoint-backup.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@ aliases: ["/tidb/dev/br-checkpoint"]

# Checkpoint Backup

Snapshot backup might be interrupted due to recoverable errors, such as disk exhaustion and node crash. Before TiDB v6.5.0, data that is backed up before the interruption would be invalidated even after the error is addressed, and you need to start the backup from scratch. For large clusters, this incurs considerable extra cost.
Snapshot backup might be interrupted due t recoverable errors, such as disk exhaustion and node crash. Before TiDB v6.5.0, data that is backed up before the interruption would be invalidated even after the error is addressed, and you need to start the backup from scratch. For large clusters, this incurs considerable extra cost.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Typo: "recoverable errors" should be "recoverable errors".1

Style Guide References

Suggested change
Snapshot backup might be interrupted due t recoverable errors, such as disk exhaustion and node crash. Before TiDB v6.5.0, data that is backed up before the interruption would be invalidated even after the error is addressed, and you need to start the backup from scratch. For large clusters, this incurs considerable extra cost.
Snapshot backup might be interrupted due to recoverable errors, such as disk exhaustion and node crash. Before TiDB v6.5.0, data that is backed up before the interruption would be invalidated even after the error is addressed, and you need to start the backup from scratch. For large clusters, this incurs considerable extra cost.

Footnotes

  1. Correct English grammar, spelling, and punctuation mistakes if any. (link) ↩


In TiDB v6.5.0, Backup & Restore (BR) introduces the checkpoint backup feature to allow continuing an interrupted backup. This feature can retain most data of the interrupted backup.
In TiDB v6.5.0, Backup & Restore (B) introduces the checkpoint backup feature to allow continuing an interrupted backup. This feature can retain most data of the interrupted backup.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

The abbreviation should be BR instead of B. This ensures consistency with the rest of the document and other documentation. Consider using Backup & Restore (BR) for the first instance, and BR for subsequent instances.

Suggested change
In TiDB v6.5.0, Backup & Restore (B) introduces the checkpoint backup feature to allow continuing an interrupted backup. This feature can retain most data of the interrupted backup.
In TiDB v6.5.0, Backup & Restore (BR) introduces the checkpoint backup feature to allow continuing an interrupted backup. This feature can retain most data of the interrupted backup.


## Application scenarios

If your TiDB cluster is large and cannot afford to back up again after a failure, you can use the checkpoint backup feature. The br command-line tool (hereinafter referred to as `br`) periodically records the shards that have been backed up. In this way, the next backup retry can use the backup progress close to the abnormal exit.
If your TiDB cluster is large and cannot affords to backup again after a failure, users can use the checkpoint backup feature. The br command-line tool (hereinafter referred to as `br`) periodically records the shards that have been backed up. In this way, the next backup retry can use the backup progress close to the abnormal exit.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Grammar: "cannot affords" should be "cannot afford".1 Also, consider using the second person ("you") when addressing users, as per the style guide. For example, "If your TiDB cluster is large and you cannot afford to back up again after a failure, you can use the checkpoint backup feature."

Style Guide References

Suggested change
If your TiDB cluster is large and cannot affords to backup again after a failure, users can use the checkpoint backup feature. The br command-line tool (hereinafter referred to as `br`) periodically records the shards that have been backed up. In this way, the next backup retry can use the backup progress close to the abnormal exit.
If your TiDB cluster is large and you cannot afford to backup again after a failure, you can use the checkpoint backup feature. The br command-line tool (hereinafter referred to as `br`) periodically records the shards that have been backed up. In this way, the next backup retry can use the backup progress close to the abnormal exit.

Footnotes

  1. Correct English grammar, spelling, and punctuation mistakes if any. (link) ↩

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Correct grammar and pronoun usage.

Fix the verb form and switch back to "you":

If your TiDB cluster is large and you cannot afford to back up again after a failure, you can use the checkpoint backup feature. The `br` command‑line tool (hereinafter referred to as `br`) periodically records the shards that have been backed up. In this way, the next backup retry can use the progress close to the abnormal exit.
🧰 Tools
🪛 LanguageTool

[grammar] ~15-~15: The modal verb ‘cannot’ requires the verb’s base form.
Context: ...f your TiDB cluster is large and cannot affords to backup again after a failure, users ...

(MD_BASEFORM)


## Implementation details

Expand All @@ -30,7 +30,7 @@ Checkpoint backup relies on the GC mechanism and cannot recover all data that ha

During the backup, `br` periodically updates the `gc-safepoint` of the backup snapshot in PD to avoid data being garbage collected. When `br` exits, the `gc-safepoint` cannot be updated in time. As a result, before the next backup retry, the data might have been garbage collected.

To avoid this situation, `br` keeps the `gc-safepoint` for about one hour by default when `gcttl` is not specified. You can set the `gcttl` parameter to extend the retention period if needed .
To avoid this situation, `br` keeps the `gc-safepoint` for about one hour by default when `gcttl` is not specified. Users can set the `gcttl` parameter to extend the retention period if needed .
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Consider using the second person ("you") when addressing users, as per the style guide. For example, "You can set the gcttl parameter to extend the retention period if needed."

Style Guide References

Suggested change
To avoid this situation, `br` keeps the `gc-safepoint` for about one hour by default when `gcttl` is not specified. Users can set the `gcttl` parameter to extend the retention period if needed .
To avoid this situation, `br` keeps the `gc-safepoint` for about one hour by default when `gcttl` is not specified. You can set the `gcttl` parameter to extend the retention period if needed .

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Remove extra space and use second‑person pronoun.

Eliminate the trailing space before the period and switch from "Users" to "you":

To avoid this situation, `br` keeps the `gc-safepoint` for about one hour by default when `gcttl` is not specified. You can set the `gcttl` parameter to extend the retention period if needed.


The following example sets `gcttl` to 15 hours (54000 seconds) to extend the retention period of `gc-safepoint`:

Expand All @@ -42,7 +42,7 @@ tiup br backup full \

> **Note:**
>
> The `gc-safepoint` created before backup is deleted after the snapshot backup is completed. You do not need to delete it manually.
> The `gc-safepoint` created before backup is deleted after the snapshot backup is completed. Users do not need to delete it manually.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Consider using the second person ("you") when addressing users, as per the style guide. For example, "You do not need to delete it manually."

Style Guide References

Suggested change
> The `gc-safepoint` created before backup is deleted after the snapshot backup is completed. Users do not need to delete it manually.
> The `gc-safepoint` created before backup is deleted after the snapshot backup is completed. You do not need to delete it manually.


### Some data needs to be backed up again

Expand Down
Loading