Open lidezhu opened 2 days ago
[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 lidezhu, ensuring that each of them provides their approval before proceeding. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.
:white_check_mark: lidezhu
:x: linkmyth
Attention: Patch coverage is 95.45455%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 57.6434%. Comparing base (
2db0922
) to head (8c17771
).
[FORMAT CHECKER NOTIFICATION]
Notice: To remove the do-not-merge/needs-linked-issue
label, please provide the linked issue number on one line in the PR body, for example: Issue Number: close #123
or Issue Number: ref #456
.
:open_book: For more info, you can check the "Contribute Code" section in the development guide.
@lidezhu: The following tests 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-mysql-test | 8c177715fcacc879639cf66182e8f79a87fb04b8 | link | true | /test cdc-integration-mysql-test |
pull-cdc-integration-kafka-test | 8c177715fcacc879639cf66182e8f79a87fb04b8 | link | true | /test cdc-integration-kafka-test |
Full PR test history. Your PR dashboard.
What problem does this PR solve?
Issue Number: close #xxx
What is changed and how it works?
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