Skip to content

cdcpb: add CDC scan priority#1493

Open
hongyunyan wants to merge 4 commits into
masterfrom
codex/cdc-scan-priority-protocol
Open

cdcpb: add CDC scan priority#1493
hongyunyan wants to merge 4 commits into
masterfrom
codex/cdc-scan-priority-protocol

Conversation

@hongyunyan

@hongyunyan hongyunyan commented Jun 23, 2026

Copy link
Copy Markdown

Background

TiCDC needs to tell downstream CDC services whether an initial scan belongs to a newly created changefeed or to an existing changefeed repair path.

Motivation

Without an explicit priority in ChangeDataRequest, downstream services cannot distinguish low-priority bootstrap scans from high-priority scans that protect existing changefeed lag.

Summary

  • Add ScanPriority to cdcpb.
  • Add ChangeDataRequest.scan_priority.
  • Add ChangeDataRequest.changefeed_id for debugging and observability.
  • Regenerate Go bindings and update the proto lock file.
  • Merge the latest master and keep the scan-priority protocol fields on top.

Validation

  • make check
  • make proto-fmt
  • make go
  • go test ./pkg/cdcpb

@ti-chi-bot

ti-chi-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown

[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 assign overvenus for approval. 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.

Details 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

Signed-off-by: hongyunyan <649330952@qq.com>
@pingcap-cla-assistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hongyunyan hongyunyan marked this pull request as ready for review July 7, 2026 06:46
@ti-chi-bot ti-chi-bot Bot requested a review from mornyx July 7, 2026 06:46
Signed-off-by: hongyunyan <649330952@qq.com>
Signed-off-by: hongyunyan <649330952@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant