Skip to content

Conversation

@toutdesuite
Copy link
Contributor

@toutdesuite toutdesuite commented Apr 21, 2020

What is changed, added or deleted? (Required)

  1. Fix grammar mistakes
  2. Unify format

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)

What is the related PR or file link(s)?

@toutdesuite toutdesuite requested a review from yikeke April 21, 2020 08:58
@toutdesuite toutdesuite added needs-cherry-pick-4.0 translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. labels Apr 21, 2020
@toutdesuite toutdesuite changed the title TiUP: Fix grammar and format for topology.example.yaml TiUP: fix grammar and format for topology.example.yaml Apr 21, 2020
@yikeke
Copy link
Contributor

yikeke commented Apr 21, 2020

Please also update the file template in quick start guide doc. @toutdesuite

@toutdesuite
Copy link
Contributor Author

@yikeke The file is updated. PTAL

log_dir: "deploy/tidb-4000/log"
numa_node: "0,1"
# Config is used to overwrite the `server_configs.tidb` values
# The following configs are used to overwrite the `server_configs.pd` values.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# The following configs are used to overwrite the `server_configs.pd` values.
# The following configs are used to overwrite the `server_configs.tidb` values.

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

@yikeke
Copy link
Contributor

yikeke commented Apr 21, 2020

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label Apr 21, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 21, 2020

/run-all-tests

@sre-bot sre-bot merged commit c49c337 into pingcap:master Apr 21, 2020
sre-bot pushed a commit to sre-bot/docs-cn that referenced this pull request Apr 21, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 21, 2020

cherry pick to release-4.0 in PR #2850

yikeke pushed a commit that referenced this pull request Apr 21, 2020
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
@toutdesuite toutdesuite deleted the modify-topology.yaml branch April 21, 2020 12:01
rleungx pushed a commit to rleungx/docs-cn that referenced this pull request May 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/can-merge Indicates a PR has been approved by a committer. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants