Skip to content
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.

Sign up for GitHub

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

Jump to bottom

*: update FAQ of tools #5031

Merged
merged 25 commits into from
Dec 21, 2020
Merged

*: update FAQ of tools #5031

merged 25 commits into from
Dec 21, 2020

Conversation

lance6716
Copy link
Contributor

First-time contributors' checklist

What is changed, added or deleted? (Required)

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

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@lance6716 lance6716 marked this pull request as draft December 4, 2020 11:15
@lance6716 lance6716 marked this pull request as ready for review December 4, 2020 11:38
@TomShawn TomShawn added needs-cherry-pick-4.0 size/small Changes of a small size. status/PTAL This PR is ready for reviewing. translation/doing This PR’s assignee is translating this PR. labels Dec 7, 2020
@lance6716
Copy link
Contributor Author

PTAL @3pointer

@TomShawn TomShawn requested review from 3pointer and removed request for overvenus December 7, 2020 03:48
overvenus
overvenus reviewed Dec 7, 2020
View reviewed changes
Copy link
Member

@overvenus overvenus 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 Dec 7, 2020
overvenus
overvenus reviewed Dec 7, 2020
View reviewed changes
ticdc/troubleshoot-ticdc.md Outdated

## TiCDC 启动任务 start-ts 与当前时间差距较大，运行过程中同步中断，出现错误 [CDC:ErrBufferReachLimit]

在 4.0.9 之后可以尝试开启 unified sorter 特性进行同步；或者可以使用 BR 工具进行一次增量备份和恢复，然后从新的时间点开启 TiCDC 同步任务。TiCDC 将会在后续版本进行优化
Copy link
Member

@overvenus overvenus Dec 7, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

Please add a link about the unified sorter feature.

Copy link
Contributor

Choose a reason for hiding this comment

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

#4959 is not merged yet.

@TomShawn TomShawn removed the status/PTAL This PR is ready for reviewing. label Dec 7, 2020
glorv
glorv reviewed Dec 10, 2020
View reviewed changes
tidb-lightning/tidb-lightning-faq.md Outdated

## TiDB Lightning 报错 `could not find first pair, this shouldn't happen`

检查日志中是否存在其他错误，并根据错误作出相应调整。比如对 `too many open files` 错误，就需要调大系统文件的描述符个数（建议 1000000 个）。
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
检查日志中是否存在其他错误，并根据错误作出相应调整。比如对 `too many open files` 错误，就需要调大系统文件的描述符个数（建议 1000000 个）。
此错误是在遍历本地排序的文件时出现异常，比较可能是由于 lightning 打开的文件数量超过系统的上。在 linux 系统中，可以使用 `ulimit -n` 命令确认此值是否过小。建议在 lightning 导入期间将此设置调整为 1000000(`ulimit -n 1000000`).

@ti-srebot
Copy link
Contributor

@overvenus, @TomShawn, @ti-srebot, @glorv, @3pointer, PTAL.

@ti-srebot
Copy link
Contributor

@lance6716, please update your pull request.

@ti-srebot
Copy link
Contributor

@, @, @, @, @, @, @, @, @, @, @, @, @, @, @3pointer, PTAL.

@ti-srebot
Copy link
Contributor

@3pointer, PTAL.

@ti-srebot
Copy link
Contributor

@lance6716, please update your pull request.

@ti-srebot
Copy link
Contributor

@3pointer, PTAL.

@ti-srebot
Copy link
Contributor

@lance6716, please update your pull request.

@ti-srebot
Copy link
Contributor

@lance6716, please update your pull request.

@Joyinqin Joyinqin mentioned this pull request Dec 21, 2020
Merged
9 tasks
@Joyinqin
Copy link
Contributor

/label translation/done

@ti-srebot ti-srebot added the translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. label Dec 21, 2020
@Joyinqin
Copy link
Contributor

/unlabel translation/doing

@ti-srebot ti-srebot removed the translation/doing This PR’s assignee is translating this PR. label Dec 21, 2020
@TomShawn TomShawn merged commit 3c0d2d8 into pingcap:master Dec 21, 2020
ti-srebot pushed a commit to ti-srebot/docs-cn that referenced this pull request Dec 21, 2020
@lance6716 @ti-srebot
cherry pick pingcap#5031 to release-4.0
01eb2df 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Dec 21, 2020
Merged
10 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #5140

TomShawn pushed a commit that referenced this pull request Dec 21, 2020
@ti-srebot @lance6716
cherry pick #5031 to release-4.0 (#5140)
0234b14 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

Co-authored-by: lance6716 <lance6716@gmail.com>
@TomShawn TomShawn added the requires-followup This PR requires a follow-up task after being merged. label Jan 5, 2021
TomShawn
TomShawn reviewed Feb 23, 2021
View reviewed changes
tidb-lightning/tidb-lightning-faq.md

## TiDB Lightning 报错 `could not find first pair, this shouldn't happen`

报错原因是遍历本地排序的文件时出现异常，可能在 lightning 打开的文件数量超过系统的上限时发生。在 linux 系统中，可以使用 `ulimit -n` 命令确认此值是否过小。建议在 lightning 导入期间将此设置调整为 1000000（`ulimit -n 1000000`）。
Copy link
Contributor

Choose a reason for hiding this comment

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

Spelling fixed in https://github.com/pingcap/docs-cn/pull/5560/files#diff-e6634f40b51d83bbcb752ea51b714154f294bc6953f372249b61c743275f8834R196.

@TomShawn TomShawn removed the requires-followup This PR requires a follow-up task after being merged. label Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glorv glorv glorv left review comments

@TomShawn TomShawn TomShawn left review comments

@overvenus overvenus overvenus approved these changes

@ti-srebot ti-srebot ti-srebot approved these changes

@3pointer 3pointer Awaiting requested review from 3pointer

Assignees

@Joyinqin Joyinqin

Labels
size/small Changes of a small size. status/LGT2 Indicates that a PR has LGTM 2. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@lance6716 @TomShawn @ti-srebot @Joyinqin @overvenus @glorv