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.

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

ticdc: add more faq #3882

Merged
merged 10 commits into from
Jul 14, 2020
Merged

ticdc: add more faq #3882

merged 10 commits into from
Jul 14, 2020

Conversation

overvenus
Copy link
Member

What is changed, added or deleted? (Required)

Add more FAQ about TiCDC.

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: N/A
  • Other reference link(s): N/A

Signed-off-by: Neil Shen <overvenus@gmail.com>
Comment on lines 170 to 173
* message.max.bytes
* replica.fetch.max.bytes
* fetch.message.max.bytes
* socket.request.max.bytes
Copy link
Member Author

@overvenus overvenus Jul 3, 2020

Choose a reason for hiding this comment

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

@amyangfei PTAL, how to tweak these params?

Signed-off-by: Neil Shen <overvenus@gmail.com>
Signed-off-by: Neil Shen <overvenus@gmail.com>
@overvenus overvenus requested a review from amyangfei July 6, 2020 12:32
Signed-off-by: Neil Shen <overvenus@gmail.com>
@TomShawn TomShawn added needs-cherry-pick-4.0 size/medium Changes of a medium size. translation/doing This PR’s assignee is translating this PR. labels Jul 6, 2020
cdc cli changefeed query --pd=http://10.0.10.25:2379 --changefeed-id 28c43ffc-2316-4f4f-a70b-d1a7c59ba79f
```

上述命令的输出中 `admin-job-type` 标志这个同步的任务的状态:
Copy link
Contributor

@amyangfei amyangfei Jul 7, 2020

Choose a reason for hiding this comment

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

不推荐用户看这个状态,建议使用最新的 cdc cli changefeed list / cdc cli changefeed query --simple 查看
ref: https://github.com/pingcap/ticdc/pull/721
ps: v4.0.3 引入

[{
"id": "4e24dde6-53c1-40b6-badf-63620e4940dc",
"summary": {
"state": "normal",
Copy link
Contributor

Choose a reason for hiding this comment

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

可以增加一下对 state 的解释
normal: 正常同步
stopped: 停止同步(手动暂停或出错)
removed: 已删除任务

Comment on lines 185 to 187
* `message.max.bytes`,将 Kafka 的 server.properties 中该参数调大到 33554432 (32 MB)。
* `replica.fetch.max.bytes`,将 Kafka 的 server.properties 中该参数调大到 33554432 (32 MB)。
* `fetch.message.max.bytes`,适当调大 comsumer.properties 中该参数,确保大于 `message.max.bytes`。
Copy link
Contributor

Choose a reason for hiding this comment

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

这 3 个值都建议大一点,可以调到 1G (1073741824)


## TiCDC 把数据同步到 Kafka 时,能在 TiDB 中控制单条消息大小的上限吗?

不能,目前 TiCDC 控制了消息大小 4MB
Copy link
Contributor

Choose a reason for hiding this comment

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

4 MB 是单个 key-value 消息的最大大小;实际每次向 kafka broker 发送的批量消息最大是 512MB

Signed-off-by: Neil Shen <overvenus@gmail.com>
@TomShawn
Copy link
Contributor

@amyangfei PTAL again

Copy link
Contributor

@amyangfei amyangfei 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
Copy link
Contributor

@amyangfei,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: docs(slack).

Copy link
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 14, 2020
@TomShawn TomShawn added require-LGT1 Indicates that the PR requires an LGTM. status/can-merge Indicates a PR has been approved by a committer. labels Jul 14, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit 91a33bd into pingcap:master Jul 14, 2020
ti-srebot pushed a commit to ti-srebot/docs-cn that referenced this pull request Jul 14, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Jul 14, 2020
5 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #3966

@overvenus overvenus deleted the cdc/faq branch July 14, 2020 09:31
TomShawn pushed a commit that referenced this pull request Jul 14, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

Co-authored-by: Neil Shen <overvenus@gmail.com>
@ran-huang ran-huang mentioned this pull request Jul 31, 2020
9 tasks
@ran-huang ran-huang added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Aug 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
require-LGT1 Indicates that the PR requires an LGTM. size/medium Changes of a medium size. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. 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.

5 participants