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

[no-release-notes] refactor sysbench_runner in preparation for pgo dolt releases #7493

Merged
merged 22 commits into from
Feb 14, 2024

Conversation

coffeegoddd
Copy link
Contributor

This PR refactors the go/performance/utils/sysbench_runner tool to reduce the duplicated code and fix the profiling support.

It renames go/performance/utils/sysbench_runner to go/performance/utils/benchmark_runner and removed go/performance/utils/tpcc_runner.

It also removes the configuration file support, as that was only used for CI testing. Now we test with go tests in go/performance/utils/benchmark_runner/run_tests.go during CI (PR for that to come).

Run is the used to run sysbench tests, RunTpcc is used to run tpcc tests.

… fixing profiling, then can refactor the rest
… profile working, need tpcc now, then unmarshal stuff
@coffeegoddd
Copy link
Contributor Author

@coffeegoddd DOLT

comparing_percentages
99.999562 to 99.999562
version result total
f75b598 not ok 26
f75b598 ok 5937431
version total_tests
f75b598 5937457
correctness_percentage
99.999562

@coffeegoddd coffeegoddd merged commit 3419237 into main Feb 14, 2024
33 of 34 checks passed
@coffeegoddd coffeegoddd deleted the db/refactor branch February 14, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant