Skip to content

Conversation

@asddongmen
Copy link
Contributor

@asddongmen asddongmen commented Nov 27, 2025

First-time contributors' checklist

What is changed, added or deleted? (Required)

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

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions (in Chinese).

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 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

Signed-off-by: dongmen <[email protected]>
@ti-chi-bot ti-chi-bot bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 27, 2025
Signed-off-by: dongmen <[email protected]>
@ti-chi-bot ti-chi-bot bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 27, 2025
@qiancai qiancai self-assigned this Nov 28, 2025
@qiancai qiancai added translation/doing This PR’s assignee is translating this PR. area/ticdc Indicates that the Issue or PR belongs to the area of TiCDC. labels Nov 28, 2025
@ti-chi-bot ti-chi-bot bot removed the missing-translation-status This PR does not have translation status info. label Nov 28, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Nov 28, 2025

@hongyunyan: adding LGTM is restricted to approvers and reviewers in OWNERS files.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

| v6.5.x - v7.1.x | 4000 |
| v7.5.x - v8.5.x | 40000 |

需要注意的是,如果你同步的表是分区表,那么每一个分区表都被 TiCDC 视为一张表,因此需要将分区表的数量也计算在内。如果你需要同步的表超过这个范围,建议你使用 TiCDC 新架构。
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.

DONE, PTAL.

Signed-off-by: dongmen <[email protected]>
@ti-chi-bot
Copy link

ti-chi-bot bot commented Nov 28, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from qiancai. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Collaborator

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


例如:当通过 changefeed 配置了过滤所有 DDL 事件后,如果上游执行 `DROP COLUMN` 操作后继续写入涉及该列的 `INSERT` 语句,TiCDC 同步这些 DML 变更到下游时,会因下游表结构不同而导致数据写入失败。

- 老架构的 TiCDC 集群同步表的数量较为有限,当一个 TiCDC 集群中同步的表数量超过建议值之后,TiCDC 可能无法正常工作,下面是具体建议数值:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- 老架构的 TiCDC 集群同步表的数量较为有限,当一个 TiCDC 集群中同步的表数量超过建议值之后,TiCDC 可能无法正常工作,下面是具体建议数值
- 对于 TiCDC [老架构](/ticdc/ticdc-classic-architecture.md),当单个 TiCDC 集群同步的表数量超过以下建议值时,TiCDC 可能无法稳定工作

| v6.5.x - v7.1.x | 4000 |
| v7.5.x - v8.5.x | 40000 |

需要注意的是,如果你同步的表是分区表,那么每一个分区表都被 TiCDC 视为一张表,因此需要将分区表的数量也计算在内。如果你需要同步的表超过这个范围,建议你使用 TiCDC 新架构,新架构 TiCDC 可以支持一个 changefeed 同步超过百万张表。[TiCDC 新架构](/ticdc/ticdc-architecture.md)
Copy link
Collaborator

@qiancai qiancai Dec 1, 2025

Choose a reason for hiding this comment

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

Suggested change
需要注意的是,如果你同步的表是分区表,那么每一个分区表都被 TiCDC 视为一张表,因此需要将分区表的数量也计算在内。如果你需要同步的表超过这个范围,建议你使用 TiCDC 新架构,新架构 TiCDC 可以支持一个 changefeed 同步超过百万张表。[TiCDC 新架构](/ticdc/ticdc-architecture.md)
> **注意:**
>
> 如果同步的是分区表,TiCDC 会将每个分区视为一张独立的表,因此在计算同步表的数量时会将分区数量也计算在内。
如果需要同步的表数量超过以上建议值,建议使用 [TiCDC 新架构](/ticdc/ticdc-architecture.md)。新架构单个 Changefeed 支持同步超过百万张表,适用于大规模同步场景。

@ti-chi-bot ti-chi-bot bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Dec 1, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Dec 1, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-12-01 08:26:12.782160961 +0000 UTC m=+251917.595938543: ☑️ agreed by qiancai.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ticdc Indicates that the Issue or PR belongs to the area of TiCDC. needs-1-more-lgtm Indicates a PR needs 1 more LGTM. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. translation/doing This PR’s assignee is translating this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants