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

Added a first pass at jmh tests for hystrix-core #750

Merged
merged 1 commit into from
Apr 11, 2015

Conversation

mattrjacobs
Copy link
Contributor

Addresses #578

@cloudbees-pull-request-builder

NetflixOSS » Hystrix » Hystrix-pull-requests #91 SUCCESS
This pull request looks good

@mattrjacobs
Copy link
Contributor Author

This is my first time using jmh, so if there are any issues with my setup/tests, I would love that feedback.

For reference, on my local machine (Mid 2012 Macbook Pro), here are my throughput numbers:

Benchmark (forceCircuitOpen) (isolationStrategy) Mode Samples Score Error Units
c.n.h.p.CommandExecutionPerfTest.baselineAsyncComputationObserve N/A N/A thrpt 5 305.506 ± 41.940 ops/ms
c.n.h.p.CommandExecutionPerfTest.baselineAsyncCustomThreadPoolObserve N/A N/A thrpt 5 236.937 ± 17.388 ops/ms
c.n.h.p.CommandExecutionPerfTest.baselineExecute N/A N/A thrpt 5 1408158.505 ± 62196.956 ops/ms
c.n.h.p.CommandExecutionPerfTest.baselineQueue N/A N/A thrpt 5 212.727 ± 11.972 ops/ms
c.n.h.p.CommandExecutionPerfTest.baselineSyncObserve N/A N/A thrpt 5 17011.994 ± 2227.406 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixExecute true THREAD thrpt 5 827.656 ± 98.067 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixExecute true SEMAPHORE thrpt 5 827.412 ± 140.393 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixExecute false THREAD thrpt 5 94.808 ± 8.571 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixExecute false SEMAPHORE thrpt 5 336.616 ± 98.050 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixObserve true THREAD thrpt 5 673.280 ± 163.860 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixObserve true SEMAPHORE thrpt 5 734.677 ± 93.178 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixObserve false THREAD thrpt 5 103.513 ± 4.732 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixObserve false SEMAPHORE thrpt 5 308.963 ± 86.085 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixQueue true THREAD thrpt 5 777.894 ± 220.728 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixQueue true SEMAPHORE thrpt 5 705.396 ± 201.299 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixQueue false THREAD thrpt 5 92.200 ± 4.534 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixQueue false SEMAPHORE thrpt 5 337.232 ± 54.647 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixToObservable true THREAD thrpt 5 772.597 ± 55.073 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixToObservable true SEMAPHORE thrpt 5 762.352 ± 64.951 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixToObservable false THREAD thrpt 5 110.178 ± 9.756 ops/ms
c.n.h.p.CommandExecutionPerfTest.hystrixToObservable false SEMAPHORE thrpt 5 312.127 ± 48.215 ops/ms

mattrjacobs added a commit that referenced this pull request Apr 11, 2015
Added a first pass at jmh tests for hystrix-core
@mattrjacobs mattrjacobs merged commit 71f23be into Netflix:master Apr 11, 2015
@mattrjacobs mattrjacobs deleted the jmh branch April 11, 2015 21:06
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