-
Notifications
You must be signed in to change notification settings - Fork 225
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
examples: add the Benchmark tool for TiKV #1331
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
…ds pattern. Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Is a go.sum missing? Do we need tests for this tool? We may also need a manual or readme for it. |
FYI, the reason why this tool is set under And as for tests, the basic and offline tests have been done for it, which can be reviewed in Meanwhile, it's acceptable to move this tool into an individual repo if we can make an agreement about this advice. |
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Signed-off-by: lucasliang <[email protected]>
Description
This pull request introduces a benchmark tool designed to evaluate the performance of a specific TiKV cluster without the necessity of deploying TiDB.
The motivation of this pr derives from the challenges associated with performance testing TiKV clusters, which typically require deploying one or more TiDB nodes. This process can be time-consuming, and issues related to performance regressions may not be promptly communicated to developers.
Therefore, this pull request offers a simplified implementation of the benchmark tool to address these concerns.
Attention
If you have any other suggestions or proposals regarding this tool, pls comment it. Welcome any advices.