Page MenuHomeSolus

initial inclusion of sysbench
AbandonedPublic

Authored by nazar on Jan 31 2022, 9:30 AM.
Tags
None
Referenced Files
F11055487: D12804.id31189.diff
Fri, Aug 11, 1:59 AM
F11055486: D12804.id31190.diff
Fri, Aug 11, 1:59 AM
F11055485: D12804.id.diff
Fri, Aug 11, 1:59 AM
F11055483: D12804.id31415.diff
Fri, Aug 11, 1:59 AM
F11038614: D12804.diff
Wed, Aug 9, 11:13 PM
F10896926: D12804.id31415.diff
Jul 2 2023, 3:13 AM
F10893596: D12804.id31190.diff
Jun 30 2023, 11:21 PM
F10844891: D12804.id31189.diff
Jun 10 2023, 10:22 PM
Subscribers

Details

Reviewers
joebonrichie
Group Reviewers
Triage Team
Maniphest Tasks
T137: Add Sysbench to Repository
Summary

resolves T137

Test Plan

sysbench cpu run

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

nazar requested review of this revision.Jan 31 2022, 9:30 AM
joebonrichie added a subscriber: joebonrichie.

Just two small nits, otherwise LGTM.

package.yml
11

nit: would prefer a slightly more descriptive description, from their github README:
"sysbench is a scriptable multi-threaded benchmark tool based on LuaJIT. It is most frequently used for database benchmarks, but can also be used to create arbitrarily complex workloads that do not involve a database server."

18

nit: we have an %autogen macro which you can use instead. With that you can drop the configure macro and pass the configure options to autogen instead.

This revision now requires changes to proceed.Feb 15 2022, 11:23 AM

With these warnings means I typically won't be able to run arc land on this cleanly

Lint was skipped when generating these changes.
Unit tests were skipped when generating these changes.

With these warnings means I typically won't be able to run arc land on this cleanly

I lost .git directory and had to upload raw diff (via phab) to fix provided remarks.

Would it work (for you) if I re-create this patch?

this patch is abandoned now, continue with D12896