Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@zier-one
Copy link
Contributor

What problem does this PR solve?

add a batch size limiter for mysql.

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test

@zier-one
Copy link
Contributor Author

/run-all-tests

1 similar comment
@zier-one
Copy link
Contributor Author

/run-all-tests

@zier-one zier-one added component/sink Sink component. status/ptal Could you please take a look? labels Mar 12, 2020
@zier-one
Copy link
Contributor Author

/run-all-tests

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 LGT1 and removed status/ptal Could you please take a look? labels Mar 12, 2020
@zier-one
Copy link
Contributor Author

/run-integration-test

@zier-one
Copy link
Contributor Author

/run-integration-tests

@zier-one
Copy link
Contributor Author

/run-all-tests

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #336   +/-   ##
=======================================
  Coverage   28.72%   28.72%           
=======================================
  Files          59       59           
  Lines        5559     5559           
=======================================
  Hits         1597     1597           
  Misses       3817     3817           
  Partials      145      145

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 863f8ea...36155a1. Read the comment docs.

@zier-one zier-one merged commit 63b1db9 into pingcap:master Mar 12, 2020
@zier-one zier-one deleted the batch_mysql branch March 12, 2020 14:28
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/sink Sink component.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants