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

config,txnkv: make txnCommitBatchSize adjustable by config #457

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

jackysp
Copy link
Contributor

@jackysp jackysp commented Mar 25, 2022

Signed-off-by: Jack Yu jackysp@gmail.com

A simple test can provide some benefits such as, for sysbench insert workload, 1pc rate increases.

txnCommitBatchSize = 16k
截屏2022-03-24 17 10 42

txnCommitBatchSize = 1M
截屏2022-03-24 17 10 48

@jackysp jackysp added the enhancement New feature or request label Mar 25, 2022
@jackysp
Copy link
Contributor Author

jackysp commented Mar 25, 2022

image

CI failed, should we upgrade go version to 1.18? @disksing

@youjiali1995
Copy link
Contributor

image

CI failed, should we upgrade go version to 1.18? @disksing

I merged a PR yesterday.

Signed-off-by: Jack Yu <jackysp@gmail.com>
@jackysp
Copy link
Contributor Author

jackysp commented Mar 28, 2022

@youjiali1995 it still failed by the same reason after I rebased master.

@youjiali1995 youjiali1995 merged commit 8489c3e into tikv:master Mar 28, 2022
qidi1 pushed a commit to qidi1/client-go that referenced this pull request Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants