-
Notifications
You must be signed in to change notification settings - Fork 678
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
reference, media: add tiflash overview doc #2155
Conversation
The image was modified once in pingcap/docs-cn#2619. Please upload the latest .jpg image. @ran-huang |
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
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.
My changed made and the rest LGTM
reference/tiflash/overview.md
Outdated
|
||
It is recommended that you deploy TiFlash in a different node from TiKV to ensure workload isolation. It is also acceptable to deploy TiFlash and TiKV in the same node, if no business isolation is required. | ||
|
||
Currently, data cannot be written directly into TiFlash. You need to write data in TiKV and then replicate it to TiFlash, because it connects to the TiDB cluster as a Learner role. TiFlash supports data replication in unit of table, but no data is replicated by default after deployment. To replicate data of a specified table, see [Create TiFlash replicas for tables](/reference/tiflash/use-tiflash.md#create-tiflash-replicas-for-tables). |
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.
Technically that is the causal relationship since learner role only accept data from other replica and cannot be operated on directly.
reference/tiflash/overview.md
Outdated
|
||
It is recommended that you deploy TiFlash in a different node from TiKV to ensure workload isolation. It is also acceptable to deploy TiFlash and TiKV in the same node, if no business isolation is required. | ||
|
||
Currently, data cannot be written directly into TiFlash. You need to write data in TiKV and then replicate it to TiFlash, because it connects to the TiDB cluster as a Learner role. TiFlash supports data replication in unit of table, but no data is replicated by default after deployment. To replicate data of a specified table, see [Create TiFlash replicas for tables](/reference/tiflash/use-tiflash.md#create-tiflash-replicas-for-tables). |
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.
Technically that is the causal relationship since learner role only accept data from other replica and cannot be operated on directly.
Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com>
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.
LGTM
Your auto merge job has been accepted, waiting for 2175 |
/merge |
Your auto merge job has been accepted, waiting for 2175 |
* reference, media: add tiflash overview doc * update pic * update reference link * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update overview.md * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update overview.md * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-authored-by: Soup <maxiaoyu@pingcap.com> 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 #2177 |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-4.0 in PR #2178 |
* reference, media: add tiflash overview doc * update pic * update reference link * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update overview.md * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update overview.md * Update reference/tiflash/overview.md Co-Authored-By: TomShawn <41534398+TomShawn@users.noreply.github.com> * Update TOC.md Co-authored-by: Soup <maxiaoyu@pingcap.com> 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: Soup <maxiaoyu@pingcap.com> Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
* cherry pick #2155 to release-4.0 Signed-off-by: sre-bot <sre-bot@pingcap.com> * resolve conflict * Update TOC.md Co-authored-by: Ran <huangran@pingcap.com> Co-authored-by: yikeke <yikeke@pingcap.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
What is changed, added or deleted? (Required)
tiflash-overview.md
.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-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.
What is the related PR or file link(s)?