Skip to content

ticdc: add description about ticdc new architecture #20597

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
Merged
merged 38 commits into from
Oct 20, 2025
Merged

ticdc: add description about ticdc new architecture #20597

merged 38 commits into from
Oct 20, 2025
+496 −125

Conversation

@qiancai
Copy link
Collaborator

@qiancai qiancai commented Mar 20, 2025 •
edited
Loading

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.4 (TiDB 8.4 versions)
  • v8.3 (TiDB 8.3 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)?

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

@qiancai qiancai added area/ticdc Indicates that the Issue or PR belongs to the area of TiCDC. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. v9.0-beta.1 This PR/issue applies to TiDB v9.0-beta.1. labels Mar 20, 2025
@ti-chi-bot ti-chi-bot bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 20, 2025
@ti-chi-bot ti-chi-bot bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 20, 2025
@qiancai qiancai self-assigned this Mar 21, 2025
@qiancai qiancai requested review from Oreoxmt and lidezhu March 21, 2025 06:06
lidezhu
lidezhu reviewed Mar 21, 2025
View reviewed changes
lidezhu and others added 2 commits March 21, 2025 18:40
@lidezhu
Update ticdc/ticdc-new-arch.md
ad09682
@qiancai @lidezhu
Apply suggestions from code review
20d8dd0 
Co-authored-by: lidezhu <47731263+lidezhu@users.noreply.github.com>
qiancai
qiancai commented Mar 25, 2025
View reviewed changes
qiancai
qiancai commented Apr 17, 2025
View reviewed changes
qiancai
qiancai commented Apr 17, 2025
View reviewed changes
Oreoxmt
Oreoxmt reviewed Apr 23, 2025
View reviewed changes
Oreoxmt
Oreoxmt reviewed Apr 23, 2025
View reviewed changes
Oreoxmt
Oreoxmt reviewed Apr 23, 2025
View reviewed changes
Oreoxmt
Oreoxmt reviewed Apr 23, 2025
View reviewed changes
@ti-chi-bot ti-chi-bot bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Apr 23, 2025
lidezhu and others added 3 commits October 17, 2025 21:19
@lidezhu @Oreoxmt
Update ticdc/ticdc-architecture.md
28124fd 
Co-authored-by: Aolin <aolinz@outlook.com>
@qiancai @Oreoxmt
Apply suggestions from code review
d81ebb8 
Co-authored-by: Aolin <aolinz@outlook.com>
@qiancai
Update monitor-ticdc.md
4e8664f
qiancai
qiancai commented Oct 18, 2025
View reviewed changes
qiancai
qiancai commented Oct 18, 2025
View reviewed changes
github-actions bot pushed a commit to qiancai/pingcap-docsite-preview that referenced this pull request Oct 18, 2025
Preview PR pingcap/docs#20597 and this preview is triggered from commit
99b0bd1 
pingcap/docs@cb0050d
benmeadowcroft
benmeadowcroft approved these changes Oct 20, 2025
View reviewed changes
Copy link
Collaborator

@benmeadowcroft benmeadowcroft left a comment

Choose a reason for hiding this comment

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

We need to add details on upgrade approaches for the new architecture, but that can be added in the subsequent round of updates

@ti-chi-bot
Copy link

ti-chi-bot bot commented Oct 20, 2025

@flowbehappy: 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.

qiancai
qiancai commented Oct 20, 2025
View reviewed changes
lidezhu
lidezhu reviewed Oct 20, 2025
View reviewed changes
@qiancai @lidezhu
Apply suggestions from code review
04f327d 
Co-authored-by: lidezhu <47731263+lidezhu@users.noreply.github.com>
@ti-chi-bot
Copy link

ti-chi-bot bot commented Oct 20, 2025

@lidezhu: 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.

@qiancai
Copy link
Collaborator Author

qiancai commented Oct 20, 2025

/approve

@qiancai qiancai added the lgtm label Oct 20, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Oct 20, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiancai

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

The pull request process is described 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

@ti-chi-bot ti-chi-bot bot added the approved label Oct 20, 2025
@lidezhu
Copy link
Contributor

lidezhu commented Oct 20, 2025

/hold

@qiancai qiancai added the needs-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. label Oct 20, 2025
@lidezhu
Copy link
Contributor

lidezhu commented Oct 20, 2025

/unhold

@ti-chi-bot ti-chi-bot bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 20, 2025
@ti-chi-bot ti-chi-bot bot merged commit b760359 into pingcap:master Oct 20, 2025
8 of 9 checks passed
@ti-chi-bot ti-chi-bot mentioned this pull request Oct 20, 2025
Merged
16 tasks
@ti-chi-bot
Copy link
Member

In response to a cherrypick label: new pull request created to branch release-8.5: #21937.

ti-chi-bot pushed a commit to ti-chi-bot/docs that referenced this pull request Oct 20, 2025
@qiancai @ti-chi-bot
ticdc: add description about ticdc new architecture (pingcap#20597)
b5e3c9e
qiancai added a commit that referenced this pull request Oct 20, 2025
@ti-chi-bot @qiancai
ticdc: add description about ticdc new architecture (#20597) (#21937)
24e1136 
Co-authored-by: Grace Cai <qqzczy@126.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benmeadowcroft benmeadowcroft benmeadowcroft approved these changes

@flowbehappy flowbehappy flowbehappy approved these changes

@lidezhu lidezhu lidezhu approved these changes

@Oreoxmt Oreoxmt Oreoxmt approved these changes

Assignees

@qiancai qiancai

Labels

approved area/ticdc Indicates that the Issue or PR belongs to the area of TiCDC. lgtm needs-1-more-lgtm Indicates a PR needs 1 more LGTM. needs-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. v9.0-beta.1 This PR/issue applies to TiDB v9.0-beta.1.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@qiancai @lidezhu @ti-chi-bot @benmeadowcroft @flowbehappy @Oreoxmt