Skip to content

ticdc: document ignore-update-only-columns filter - #21883

Open
lidezhu wants to merge 3 commits into
masterfrom
cdc/add-ignore-config
Open

ticdc: document ignore-update-only-columns filter#21883
lidezhu wants to merge 3 commits into
masterfrom
cdc/add-ignore-config

Conversation

@lidezhu

@lidezhu lidezhu commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

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)

What is the related PR or file link(s)?

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

AI agent involvement

  • The changes in this PR were primarily made by an AI agent on behalf of the PR author.

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

Summary by CodeRabbit

  • 文档
    • 新增 ignore-update-only-columns 配置说明,用于过滤仅更新指定列的 UPDATE 事件。
    • 明确该配置仅对 Kafka 下游生效;涉及主键或唯一键变更的事件不会被过滤。
    • 补充配置版本、大小写匹配、无效列处理及非 Kafka 下游行为说明。
    • 在 OpenAPI v2 的创建、查询和更新示例中新增对应配置字段。

@ti-chi-bot ti-chi-bot Bot added missing-translation-status This PR does not have translation status info. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b1251106-5c7d-4718-9910-a6e42c4a50b9

📥 Commits

Reviewing files that changed from the base of the PR and between a54d85f and f4a53e5.

📒 Files selected for processing (3)
  • ticdc/ticdc-changefeed-config.md
  • ticdc/ticdc-filter.md
  • ticdc/ticdc-open-api-v2.md

Included review availability: Your plan includes up to 4 reviews per rolling hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

文档新增 ignore-update-only-columns 配置,说明其 Kafka 下游行为、过滤规则和例外条件,并在 OpenAPI v2 的创建、查询及更新示例中加入对应字段。

Changes

UPDATE 列过滤配置

Layer / File(s) Summary
事件过滤配置说明
ticdc/ticdc-changefeed-config.md, ticdc/ticdc-filter.md
新增 ignore-update-only-columns 配置及示例。文档说明 Kafka 下游限制、列变更过滤规则、主键或唯一键变更例外、大小写匹配、无效列处理和非 Kafka 下游行为。
OpenAPI v2 字段同步
ticdc/ticdc-open-api-v2.md
在同步任务创建、查询响应和更新请求示例中加入 ignore_update_only_columns,并补充参数说明。

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🔵 Low · up to f4a53

The documentation for ignore_update_only_columns does not describe the primary-key and unique-key exception or clarify that INSERT and DELETE events are unaffected. This is a bounded documentation risk and the PR is mergeable with explicit owner follow-up to correct the description.

Suggested labels: requires-version-specific-changes

Suggested reviewers: 3aceshowhand

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning PR 描述未填写“变更内容”必填部分,缺少本次文档变更的具体说明和原因。 补充变更内容及原因,并确认 master 和 v8.5 版本选择与实际变更范围一致。
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 标题准确概括了为 TiCDC 文档补充 ignore-update-only-columns 过滤器说明这一主要变更。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cdc/add-ignore-config

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f3c77922-f9b6-4a22-988f-4adcbf4d0d05

📥 Commits

Reviewing files that changed from the base of the PR and between 26f202b and 43836f8.

📒 Files selected for processing (3)
  • ticdc/ticdc-changefeed-config.md
  • ticdc/ticdc-filter.md
  • ticdc/ticdc-open-api-v2.md

Comment thread ticdc/ticdc-open-api-v2.md Outdated
@lidezhu
lidezhu requested a review from lilin90 August 13, 2026 09:04
@lilin90 lilin90 added the translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. label Aug 13, 2026
@ti-chi-bot ti-chi-bot Bot removed the missing-translation-status This PR does not have translation status info. label Aug 13, 2026
@lilin90 lilin90 added for-release This PR relates to a TiDB release but does not have a tracked feature. and removed for-release This PR relates to a TiDB release but does not have a tracked feature. labels Aug 13, 2026
@lilin90 lilin90 self-assigned this Aug 13, 2026
@lilin90 lilin90 added area/ticdc Indicates that the Issue or PR belongs to the area of TiCDC. needs-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Aug 13, 2026
Comment thread ticdc/ticdc-changefeed-config.md Outdated
Comment thread ticdc/ticdc-filter.md Outdated
Comment thread ticdc/ticdc-open-api-v2.md Outdated
@ti-chi-bot

ti-chi-bot Bot commented Aug 18, 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 ask for approval from lilin90. 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

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-cherry-pick-release-8.5 Should cherry pick this PR to release-8.5 branch. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. 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.

2 participants