Open sdojjy opened 6 days ago
/retest-required
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 57.6802%. Comparing base (
401b2bc
) to head (cee0a1d
). Report is 2 commits behind head on master.
@kevin-xianliu: adding LGTM is restricted to approvers and reviewers in OWNERS files.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: 3AceShowHand, asddongmen, kevin-xianliu
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Timeline:
2024-06-27 02:50:15.636285726 +0000 UTC m=+860742.121774558
: :ballot_box_with_check: agreed by 3AceShowHand.2024-06-28 01:59:29.4900454 +0000 UTC m=+944095.975534238
: :ballot_box_with_check: agreed by asddongmen./retest-required
/retest-required
/retest-required
@sdojjy: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-cdc-integration-kafka-test | cee0a1d272a7f523c44b717ba1cea004ad9b5fb9 | link | true | /test cdc-integration-kafka-test |
Full PR test history. Your PR dashboard.
What problem does this PR solve?
Issue Number: close #11354
What is changed and how it works?
The below
cdc-v2
configuration is not used anymore, and ticdc will implement the new architecture in another way, so remove it to reduce the codebase.furthermore, it's a debug configuration, so we can remove it without breaking the compatibility
Check List
Tests
Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note