Skip to content
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.

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

*: support to replicate tables without explicit row id #1005

Merged
merged 7 commits into from
Oct 20, 2020

Conversation

amyangfei
Copy link
Contributor

What problem does this PR solve?

Close https://github.com/pingcap/ticdc/issues/994

What is changed and how it works?

  • Add a force-replicate option in changefeed config, if both enable-old-value and force-replicate are true, TiCDC will replicate tables without any explicit row id.
  • Fix a bug in the Clone function of schemaSnapshot.
  • Pass a forceReplicate option in IsEligible funcion of a TableInfo to control the eligible logic.

Check List

Tests

  • Unit test
  • Integration test

Code changes

  • Has exported function/method change

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation

Release note

  • Support to replicate tables without explicit row id

@amyangfei amyangfei added the needs-cherry-pick-release-4.0 Should cherry pick this PR to release-4.0 branch. label Oct 19, 2020
@amyangfei amyangfei added this to the v4.0.8 milestone Oct 19, 2020
@amyangfei
Copy link
Contributor Author

/run-all-tests

@overvenus
Copy link
Member

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Oct 19, 2020
@amyangfei
Copy link
Contributor Author

/run-all-tests

@amyangfei amyangfei added the status/ptal Could you please take a look? label Oct 19, 2020
@ti-srebot ti-srebot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Oct 20, 2020
@zier-one
Copy link
Contributor

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Oct 20, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot
Copy link
Contributor

@amyangfei merge failed.

@zier-one
Copy link
Contributor

/run-integraion-tests

2 similar comments
@amyangfei
Copy link
Contributor Author

/run-integraion-tests

@amyangfei
Copy link
Contributor Author

/run-integraion-tests

@amyangfei
Copy link
Contributor Author

/merge

@ti-srebot
Copy link
Contributor

/run-all-tests

@codecov-io
Copy link

Codecov Report

Merging #1005 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master      #1005   +/-   ##
===========================================
  Coverage   32.2260%   32.2260%           
===========================================
  Files           101        101           
  Lines         11680      11680           
===========================================
  Hits           3764       3764           
  Misses         7503       7503           
  Partials        413        413           

@amyangfei
Copy link
Contributor Author

/merge

@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit fea8a70 into pingcap:master Oct 20, 2020
ti-srebot pushed a commit to ti-srebot/ticdc that referenced this pull request Oct 20, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #1010

@amyangfei amyangfei deleted the replicate-table-without-key branch October 20, 2020 10:50
zier-one pushed a commit that referenced this pull request Oct 23, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

Co-authored-by: amyangfei <amyangfei@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-cherry-pick-release-4.0 Should cherry pick this PR to release-4.0 branch. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. status/ptal Could you please take a look?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Let TiCDC replicate table without primary/unique key
5 participants