Skip to content
This repository has been archived by the owner on Jan 3, 2025. It is now read-only.

zh: add document about how to do performance test #85

Merged
merged 25 commits into from
May 26, 2020

Conversation

WangXiangUSTC
Copy link
Contributor

@WangXiangUSTC WangXiangUSTC commented May 20, 2020

What is changed, added or deleted? (Required)

add document about how to do the performance test, most content is copy from https://pingcap.com/docs-cn/tidb-data-migration/stable/benchmark-v1.0-ga/ and then update

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

  • master (the latest development version, including v2.0 changes for now)

  • v1.0 (TiDB DM 1.0 versions)

  • Other reference link(s):

@WangXiangUSTC WangXiangUSTC added the special-week PR from Document Special Week. label May 20, 2020
@TomShawn TomShawn added the translation/welcome Waits for a contributor to translate this PR to docs-dm/zh or docs-dm/en. label May 20, 2020
@csuzhangxc
Copy link
Member

most content is copy from https://pingcap.com/docs-cn/tidb-data-migration/stable/benchmark-v1.0-ga/

@WangXiangUSTC 我们更新一下 benchmark-v1.0-ga 吗?(比如删除里面一部分内容、然后 link 到这个文档之类的?)

@WangXiangUSTC
Copy link
Contributor Author

most content is copy from https://pingcap.com/docs-cn/tidb-data-migration/stable/benchmark-v1.0-ga/

@WangXiangUSTC 我们更新一下 benchmark-v1.0-ga 吗?(比如删除里面一部分内容、然后 link 到这个文档之类的?)

可以的,我在这个 pr 里更新下


## 部署测试环境

- 使用 TiDB Ansible 部署 TiDB 测试集群,所有配置使用 TiDB Ansible 提供的默认配置。
Copy link
Contributor

@lichunzhu lichunzhu May 20, 2020

Choose a reason for hiding this comment

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

是不是让用户拿 tiup 部署比较好,ansible 4.0 后已经不是官方文档推荐的方式了,贴上 tiup 部署的 link

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tiup 部署 dm 的文档现在还没有,等后面有了再改下这部分内容吧

Copy link
Contributor

Choose a reason for hiding this comment

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

这句话不是让用户用 Ansible 部署 TiDB 集群吗?TiUP 已经支持 TiDB 了吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

对,改成 tiup 了


### 获取测试结果

通过 `query-status` 命令观测 DM 的同步状态,通过 Grafana 观测 DM 的监控指标。主要包括同步延迟 `replicate lag`,单位时间内完成的 job 数量 `finished sqls jobs` 等。
Copy link
Contributor

Choose a reason for hiding this comment

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

需要说一下怎么弄 Grafana 吗?

Copy link
Member

@csuzhangxc csuzhangxc May 21, 2020

Choose a reason for hiding this comment

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

replicate lag 在这个任务配置里应该看不到,因为没有启用 heartbeat。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

更新了配置,加上了 enable-heartbeat

Copy link
Contributor Author

Choose a reason for hiding this comment

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

需要说一下怎么弄 Grafana 吗?

暂时还没有专门介绍怎么部署 grafana 的文档,都是 ansible 自动部署上的,后面有的话再加个链接吧

@sre-bot sre-bot added the status/PTAL This PR is ready for reviewing. label May 21, 2020
@yikeke yikeke requested review from miaoqingli and removed request for kissmydb May 21, 2020 03:59
@lilin90 lilin90 requested review from toutdesuite and removed request for lilin90 May 21, 2020 07:27
WangXiangUSTC and others added 3 commits May 21, 2020 18:03
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Copy link
Member

@csuzhangxc csuzhangxc 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


#### 创建数据同步任务

1. 创建上游 MySQL 的 source,将 `source-id` 配置为 `source-1`。
Copy link

@miaoqingli miaoqingli May 22, 2020

Choose a reason for hiding this comment

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

感觉可以增加一个新增 source 方法的链接

Copy link
Contributor Author

Choose a reason for hiding this comment

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

加了链接


#### 获取测试结果

通过 `query-status` 命令观测 DM 的同步状态,通过 Grafana 观测 DM 的监控指标。主要包括同步延迟 `replicate lag`,单位时间内完成的 job 数量 `finished sqls jobs` 等。

Choose a reason for hiding this comment

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

增加性能定位的文章链接

Copy link
Contributor Author

Choose a reason for hiding this comment

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

加了链接

@toutdesuite toutdesuite added the status/require-change The author needs to address comments. label May 22, 2020
@miaoqingli
Copy link

LGTM

@yikeke yikeke added translation/doing This PR’s assignee is translating this PR. and removed translation/welcome Waits for a contributor to translate this PR to docs-dm/zh or docs-dm/en. labels May 22, 2020
Copy link
Contributor

@toutdesuite toutdesuite 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


#### 创建数据同步任务

1. 创建上游 MySQL 的 source,将 `source-id` 配置为 `source-1`。详细操作方法参考:[加载数据源配置](manage-replication-tasks.md#加载数据源配置)。
Copy link
Contributor

Choose a reason for hiding this comment

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

加载数据源配置 这篇文档已经删除了,可以看看链接到这篇是否合适

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的,等那个 pr 合并了我再更新这里

Copy link
Contributor

Choose a reason for hiding this comment

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

@WangXiangUSTC #90 has been merged, please update links and resolve conflict~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

链接改了

threads: 32
```

创建数据同步任务的详细操作参考[创建数据同步任务]((manage-replication-tasks.md#创建数据同步任务)。
Copy link
Contributor

Choose a reason for hiding this comment

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

这里链接到这篇是否合适

Copy link
Contributor Author

Choose a reason for hiding this comment

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

链接改了


#### 创建数据同步任务

1. 创建上游 MySQL 的 source, source-id 配置为 `source-1`。(如果在全量同步性能测试中已经创建,则不需要再次创建)。详细操作方法参考:[加载数据源配置](manage-replication-tasks.md#加载数据源配置)。
Copy link
Contributor

Choose a reason for hiding this comment

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

这里链接也要改,同理下面涉及到的相关链接也要改~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

链接改了

WangXiangUSTC and others added 2 commits May 23, 2020 23:52
@WangXiangUSTC WangXiangUSTC added status/LGT2 Indicates that a PR has LGTM 2. and removed status/PTAL This PR is ready for reviewing. labels May 25, 2020
@WangXiangUSTC
Copy link
Contributor Author

@toutdesuite PTAL again

Co-authored-by: toutdesuite <guizhiluo2014@163.com>
@WangXiangUSTC WangXiangUSTC merged commit d451066 into docs-special-week May 26, 2020
@WangXiangUSTC WangXiangUSTC deleted the xiang/performance_test branch May 26, 2020 06:25
lilin90 added a commit that referenced this pull request May 27, 2020
* scripts: add a script to run markdownlint check locally (#74)

* Update ci.yaml (#84)

* zh: update TOC (#75)

* update TOC

* Update TOC.md

* Update TOC.md

* Update zh/TOC.md

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update zh/TOC.md

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* address comments

* fix dead link and address comment

* Apply suggestions from code review

* fix levels

* Update links in TOC

* fix CI

* remove link for 测试验证

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: yikeke <yikeke@pingcap.com>

* zh: add the requirement of software and hardware (#76)

* add requirement

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* update word

* address comemnt

* Apply suggestions from code review

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* update disk requirement

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* rename file name

* Apply suggestions from code review

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* address comment

Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* zh: update documents about error handling (#78)

* zh: add a document about performance handling (#81)

* zh: add document about creating task and verify (#77)

* zh: add quick start doc for dm deployment (#80)

* zh: update docs about key features (#86)

* zh: add a document about how to optimize config (#89)

* add config optimize

* rename file

* Apply suggestions from code review

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>

* format

* address comment

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/tune-configuration.md

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/tune-configuration.md

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>

* zh: Further update TOC.md (#92)

* Update TOC.md

* add omitted links

* update format

* Update TOC.md

* Update TOC.md

* Update zh/TOC.md

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* update file name

* add alias

* fix dead links

* Revert "fix dead links"

This reverts commit 9cf7b75.

* fix dead links in zh

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* zh: update dm-upgrade (#87)

* zh: add a document about scenarios (#88)

* zh: update daily-check.md (#91)

* update daily-check.md

* address comment

* address comment

* Update zh/daily-check.md

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>

* zh: update docs about alert rules and handling (#93)

* zh: add a doc about command-line-flags and update TOC (#94)

* zh: add a doc about command-line-flags and update TOC

* zh: update TOC

* command-line-flags: address comments

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* command-line-flags: address comments

* remove empty files

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* zh: add files about manage task (#90)

* add task manage for special week

* minor change

* rename file

* rename file

* minor change

* minor change

* fix CI

* fix CI

* address comment

* minor change

* Update skip-or-replace-abnormal-sql-statements.md

* Update zh/create-task.md

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>

* address comment

* address comment

* update TOC

* address comment

* Update TOC.md

* add query-status and query-error in TOC

* fix toc

* Update zh/query-error.md

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* Update zh/query-status.md

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* rm manage-replication-tasks.md

* fix CI

* fix CI

* address comment

* fix bug

* update TOC

* add dmctl-intro.md

* Update zh/dmctl-intro.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* rename file

* Update TOC.md

* address comment

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>

* zh: add document about how to do performance test (#85)

* zh: update TOC and remove empty files (#95)

* update TOC and remove empty files

* Update TOC.md

Co-authored-by: yikeke <yikeke@pingcap.com>

* zh: update release notes in toc

* zh: update wording

* *: make image name unique at the repo level (#96)

* *: make image name unique at repo level

* en: fix one link

Co-authored-by: Ran <huangran@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: yikeke <yikeke@pingcap.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>
Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>
Co-authored-by: GMHDBJD <35025882+GMHDBJD@users.noreply.github.com>
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@toutdesuite toutdesuite added translation/done This PR has been translated. and removed translation/doing This PR’s assignee is translating this PR. labels Jul 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
special-week PR from Document Special Week. status/LGT2 Indicates that a PR has LGTM 2. status/require-change The author needs to address comments. translation/done This PR has been translated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants