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

zh, en: update error-handling.md#178

Merged
ti-srebot merged 6 commits into
pingcap:masterfrom
toutdesuite:error-handling-doc
Jul 31, 2020
Merged

zh, en: update error-handling.md#178
ti-srebot merged 6 commits into
pingcap:masterfrom
toutdesuite:error-handling-doc

Conversation

@toutdesuite
Copy link
Copy Markdown
Contributor

@toutdesuite toutdesuite commented Jul 24, 2020

What is changed, added or deleted? (Required)

  • en: update error-handling.md
  • zh: remove reasons for DM 的全量数据导入处理模块不支持对 dump 处理模块导出的 SQL 文件进行切分。

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

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

@toutdesuite toutdesuite added the translation/from-zh This PR is translated from a PR in docs-dm/zh. label Jul 24, 2020
@toutdesuite toutdesuite requested review from TomShawn and removed request for yikeke July 27, 2020 03:09
Comment thread en/error-handling.md Outdated
Comment thread en/error-handling.md Outdated
Comment thread en/error-handling.md
Comment thread en/error-handling.md Outdated
Comment thread en/error-handling.md Outdated
Comment thread en/error-handling.md Outdated
toutdesuite and others added 3 commits July 28, 2020 17:19
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@toutdesuite
Copy link
Copy Markdown
Contributor Author

@WangXiangUSTC PTAL, thanks~

@toutdesuite toutdesuite changed the title en: update error-handling.md zh, en: update error-handling.md Jul 29, 2020
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 29, 2020
@yikeke yikeke requested a review from csuzhangxc July 30, 2020 09:02
@yikeke yikeke added the status/PTAL This PR is ready for reviewing. label Jul 30, 2020
Copy link
Copy Markdown
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.

LGTM

@ti-srebot ti-srebot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jul 31, 2020
@yikeke
Copy link
Copy Markdown
Contributor

yikeke commented Jul 31, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jul 31, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit 5f6e2ca into pingcap:master Jul 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. status/PTAL This PR is ready for reviewing. translation/from-zh This PR is translated from a PR in docs-dm/zh.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants