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

Add benchmarks for all client commands #10

Merged
merged 2 commits into from
Apr 10, 2017

Conversation

nastevens
Copy link
Contributor

Adds benchmarks for all client commands. Since the Bencher type is unstable in Rust (as of 1.16.0), the benchmarks are placed behind an unstable feature flag.

Benchmarks may be easily run using rustup:

rustup run nightly cargo bench --features=unstable

Preliminary benchmark results on a 2015 MacBook Pro (2.5 GHz Intel Core
i7):

test bench::bench_decr      ... bench:     137,422 ns/iter (+/- 317,266)
test bench::bench_event     ... bench:     157,708 ns/iter (+/- 279,659)
test bench::bench_gauge     ... bench:     138,378 ns/iter (+/- 254,691)
test bench::bench_histogram ... bench:     157,012 ns/iter (+/- 308,771)
test bench::bench_incr      ... bench:     183,358 ns/iter (+/- 305,843)
test bench::bench_set       ... bench:     177,160 ns/iter (+/- 308,811)
test bench::bench_timing    ... bench:     163,766 ns/iter (+/- 293,362)

Adds benchmarks for all client commands. Since the `Bencher` type is
unstable in Rust (as of 1.16.0), the benchmarks are placed behind an
`unstable` feature flag.

Benchmarks may be easily run using rustup:

    rustup run nightly cargo bench --features=unstable

Preliminary benchmark results on a 2015 MacBook Pro (2.5 GHz Intel Core
i7):

    test bench::bench_decr      ... bench:     137,422 ns/iter (+/- 317,266)
    test bench::bench_event     ... bench:     157,708 ns/iter (+/- 279,659)
    test bench::bench_gauge     ... bench:     138,378 ns/iter (+/- 254,691)
    test bench::bench_histogram ... bench:     157,012 ns/iter (+/- 308,771)
    test bench::bench_incr      ... bench:     183,358 ns/iter (+/- 305,843)
    test bench::bench_set       ... bench:     177,160 ns/iter (+/- 308,811)
    test bench::bench_timing    ... bench:     163,766 ns/iter (+/- 293,362)

Signed-off-by: Nick Stevens <[email protected]>
@nastevens nastevens mentioned this pull request Apr 10, 2017
@mcasper
Copy link
Owner

mcasper commented Apr 10, 2017

@nastevens Thanks for this!

@mcasper mcasper merged commit 131a32f into mcasper:master Apr 10, 2017
@nastevens nastevens deleted the add-benchmarks branch April 26, 2017 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants