-
Notifications
You must be signed in to change notification settings - Fork 286
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
etcd_worker: batch etcd patch (#3277) #3390
etcd_worker: batch etcd patch (#3277) #3390
Conversation
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
/run-unit-tests |
/run-integration-tests |
/run-integration-tests |
Codecov Report
@@ Coverage Diff @@
## release-4.0 #3390 +/- ##
===================================================
+ Coverage 53.7661% 57.0452% +3.2791%
===================================================
Files 153 170 +17
Lines 15958 19758 +3800
===================================================
+ Hits 8580 11271 +2691
- Misses 6475 7427 +952
- Partials 903 1060 +157 |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 8588ac6
|
This is an automated cherry-pick of #3277
This PR also include: https://github.com/pingcap/ticdc/pull/3681
What problem does this PR solve?
This PR is an improvement to the issue's(#3112 ) fix.
What is changed and how it works?
patches
as possible into a batch and submit them as an etcd transaction, which can reduce the etcd qps.2. Get the response from etcdwatchCh
as much as possible to reduce the number of etcd worker tick reactor.Check List
Tests
Side effects
Related changes
Release note