Skip to content

tiflash: add tiflash configuration document#2188

Merged
ran-huang merged 5 commits into
pingcap:masterfrom
TomShawn:tiflash-config-scale
Apr 8, 2020
Merged

tiflash: add tiflash configuration document#2188
ran-huang merged 5 commits into
pingcap:masterfrom
TomShawn:tiflash-config-scale

Conversation

@TomShawn
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn commented Apr 7, 2020

What is changed, added or deleted? (Required)

Add TiFlash configuration document.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

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)?

@TomShawn TomShawn added translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. needs-cherry-pick-3.1 labels Apr 7, 2020
@TomShawn TomShawn requested review from ilovesoup and yikeke April 7, 2020 10:32
@yikeke yikeke assigned ran-huang and unassigned ran-huang Apr 7, 2020
@yikeke yikeke requested a review from ran-huang April 7, 2020 10:36
Comment thread reference/tiflash/configuration.md Outdated
Comment thread reference/tiflash/configuration.md Outdated
Comment thread reference/tiflash/configuration.md Outdated
service_addr = The listening address of TiFlash Raft services and coprocessor services.
```

Multiple TiFlash nodes elect a master to add or delete placement rules to PD and you need three parameters to control this process.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Multiple TiFlash nodes elect a master to add or delete placement rules to PD and you need three parameters to control this process.
Multiple TiFlash nodes elect a master to add or delete placement rules to PD, and you need three parameters to control this process.

Comment thread reference/tiflash/configuration.md Outdated
Comment thread reference/tiflash/configuration.md Outdated
Copy link
Copy Markdown
Contributor

@ilovesoup ilovesoup left a comment

Choose a reason for hiding this comment

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

LGTM

@TomShawn
Copy link
Copy Markdown
Contributor Author

TomShawn commented Apr 8, 2020

@ran-huang PTAL again, thanks!

Copy link
Copy Markdown
Contributor

@ran-huang ran-huang left a comment

Choose a reason for hiding this comment

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

Rest LGTM

Comment thread reference/tiflash/configuration.md Outdated
Comment thread reference/tiflash/configuration.md Outdated
Co-Authored-By: Ran <huangran@pingcap.com>
@ran-huang ran-huang merged commit 3dfa4b3 into pingcap:master Apr 8, 2020
sre-bot pushed a commit to sre-bot/docs that referenced this pull request Apr 8, 2020
* tiflash: add tiflash configuration document

* address comments

* Update configuration.md

* Apply suggestions from code review

Co-Authored-By: Ran <huangran@pingcap.com>

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Ran <huangran@pingcap.com>
@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Apr 8, 2020

cherry pick to release-3.1 in PR #2196

@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Apr 8, 2020

cherry pick to release-4.0 in PR #2197

sre-bot pushed a commit to sre-bot/docs that referenced this pull request Apr 8, 2020
* tiflash: add tiflash configuration document

* address comments

* Update configuration.md

* Apply suggestions from code review

Co-Authored-By: Ran <huangran@pingcap.com>

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Ran <huangran@pingcap.com>
@TomShawn TomShawn deleted the tiflash-config-scale branch April 8, 2020 06:16
yikeke pushed a commit that referenced this pull request Apr 8, 2020
* tiflash: add tiflash configuration document

* address comments

* Update configuration.md

* Apply suggestions from code review

Co-Authored-By: Ran <huangran@pingcap.com>

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Ran <huangran@pingcap.com>

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Ran <huangran@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants