-
Notifications
You must be signed in to change notification settings - Fork 709
tiflash: add scale.md doc #2184
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.
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
Conversation
reference/tiflash/scale.md
Outdated
| > | ||
| > The offline process described in this section does not delete the data on the offline node. If you need to take the node online again, delete the data manually. | ||
|
|
||
| 1. Take the node to be scaled in offline. See [Take TiFlash nodes offline](/reference/tiflash/maintain.md#take-tiflash-nodes-offline) for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reference anchor is not finalized since the translation of maintain.md is not finished yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI @toutdesuite
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ran-huang The anchor is decided: #take-a-tiflash-node-down.
ilovesoup
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This applies only to 3.1.
4.0 will have different content.
The 4.0 changes might refer to pingcap/docs-cn#2641. |
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
|
@TomShawn PTAL again~ |
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
TomShawn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rest LGTM
|
@TomShawn PTAL |
|
I will merge this PR after #2189 is merged. |
* tiflash: add scale.md doc * Apply suggestions from code review Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * minor update * update anchor Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
|
cherry pick to release-3.1 in PR #2209 |
|
A dead link is introduced by this PR but it will be fixed by #2189 soon. |
* tiflash: add scale.md doc * Apply suggestions from code review Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * minor update * update anchor Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
|
cherry pick to release-4.0 in PR #2210 |
* tiflash: add scale.md doc * Apply suggestions from code review Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * minor update * update anchor Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com> Co-authored-by: Ran <huangran@pingcap.com> Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
* tiflash: add scale.md doc * Apply suggestions from code review Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * minor update * update anchor Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com> Co-authored-by: Ran <huangran@pingcap.com> Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
What is changed, added or deleted? (Required)
Add
scale.mddoc for TiFlash.Which TiDB version(s) do your changes apply to? (Required)
If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-4.0, needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.
What is the related PR or file link(s)?