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

kvclient: support resolve lock for ticdc #459

Merged
merged 9 commits into from
Apr 16, 2020

Conversation

zier-one
Copy link
Contributor

What problem does this PR solve?

support resolve lock for ticdc

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test

@zier-one zier-one added status/ptal Could you please take a look? component/kv-client TiKV kv log client component. labels Apr 13, 2020
@zier-one zier-one requested a review from amyangfei April 13, 2020 13:08
@zier-one
Copy link
Contributor Author

/run-all-tests

@codecov-io
Copy link

codecov-io commented Apr 13, 2020

Codecov Report

Merging #459 into master will increase coverage by 0.1044%.
The diff coverage is 22.5645%.

@@               Coverage Diff                @@
##             master       #459        +/-   ##
================================================
+ Coverage   29.2417%   29.3461%   +0.1044%     
================================================
  Files            59         58         -1     
  Lines          5328       5827       +499     
================================================
+ Hits           1558       1710       +152     
- Misses         3629       4000       +371     
+ Partials        141        117        -24     

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

3 similar comments
@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

2 similar comments
@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

cdc/kv/client.go Outdated Show resolved Hide resolved
cdc/kv/client.go Outdated Show resolved Hide resolved
cdc/kv/client.go Outdated Show resolved Hide resolved
cdc/kv/client.go Outdated Show resolved Hide resolved
leoppro and others added 3 commits April 15, 2020 15:25
Co-Authored-By: MyonKeminta <9948422+MyonKeminta@users.noreply.github.com>
@zier-one
Copy link
Contributor Author

/run-all-tests

Copy link
Contributor

@MyonKeminta MyonKeminta left a comment

Choose a reason for hiding this comment

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

LGTM

@zier-one
Copy link
Contributor Author

/run-kafka-tests

@MyonKeminta
Copy link
Contributor

By the way I think it would be good if there can be a test that checks the resolve lock won't kill an ongoing transaction..

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one zier-one added LGT1 and removed status/ptal Could you please take a look? labels Apr 16, 2020
Copy link
Contributor

@amyangfei amyangfei left a comment

Choose a reason for hiding this comment

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

LGTM

@amyangfei amyangfei added LGT2 and removed LGT1 labels Apr 16, 2020
@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-integration-tests

1 similar comment
@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one zier-one merged commit 5987945 into pingcap:master Apr 16, 2020
@zier-one zier-one deleted the resolve_lock branch April 16, 2020 09:41
5kbpers pushed a commit to 5kbpers/ticdc that referenced this pull request Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/kv-client TiKV kv log client component.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants