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.

Sign up for GitHub

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

Jump to bottom

Minor refactoring #316

Merged
merged 1 commit into from
Aug 9, 2020
Merged

Minor refactoring #316

merged 1 commit into from
Aug 9, 2020

Conversation

barambani
Copy link
Member

Just refactoring, it doesn't bring anything in the performances.

Reference (master)
Benchmark                                               Mode  Cnt      Score     Error  Units
ParallelLoadLaserdiscCatsBench.parallelLoad1           thrpt  200   5291.802 ±  14.365  ops/s
ParallelLoadLaserdiscCatsBench.parallelLoad2           thrpt  200  13374.111 ± 117.475  ops/s
ParallelLoadLaserdiscCatsRespBench.parallelLoad1       thrpt  200   5951.609 ±  56.826  ops/s
ParallelLoadLaserdiscCatsRespBench.parallelLoad2       thrpt  200  10264.231 ±  61.130  ops/s
ParallelLoadLaserdiscCatsBitVectorBench.parallelLoad1  thrpt  200   7296.143 ±  33.001  ops/s
ParallelLoadLaserdiscCatsBitVectorBench.parallelLoad2  thrpt  200  11417.866 ± 120.308  ops/s
ParallelLoadLaserdiscCatsByteBench.parallelLoad1       thrpt  200  48985.944 ± 478.882  ops/s
ParallelLoadLaserdiscCatsByteBench.parallelLoad2       thrpt  200  52861.300 ± 143.582  ops/s
ParallelLoadLaserdiscCatsBaselineBench.parallelLoad1   thrpt  200  49127.852 ± 349.831  ops/s
ParallelLoadLaserdiscCatsBaselineBench.parallelLoad2   thrpt  200  53043.959 ± 145.580  ops/s

This Branch:
Benchmark                                               Mode  Cnt      Score     Error  Units
ParallelLoadLaserdiscCatsBench.parallelLoad1           thrpt  200   5173.993 ±  21.867  ops/s
ParallelLoadLaserdiscCatsBench.parallelLoad2           thrpt  200  13530.416 ±   5.406  ops/s
ParallelLoadLaserdiscCatsRespBench.parallelLoad1       thrpt  200   5892.701 ±  58.980  ops/s
ParallelLoadLaserdiscCatsRespBench.parallelLoad2       thrpt  200  10474.042 ±  73.011  ops/s
ParallelLoadLaserdiscCatsBitVectorBench.parallelLoad1  thrpt  200   7162.177 ±  31.172  ops/s
ParallelLoadLaserdiscCatsBitVectorBench.parallelLoad2  thrpt  200  11343.853 ± 111.668  ops/s
ParallelLoadLaserdiscCatsByteBench.parallelLoad1       thrpt  200  48806.660 ± 368.138  ops/s
ParallelLoadLaserdiscCatsByteBench.parallelLoad2       thrpt  200  52606.223 ± 150.208  ops/s
ParallelLoadLaserdiscCatsBaselineBench.parallelLoad1   thrpt  200  49472.836 ± 165.165  ops/s
ParallelLoadLaserdiscCatsBaselineBench.parallelLoad2   thrpt  200  52775.279 ± 170.368  ops/s

@barambani barambani requested a review from sirocchj as a code owner August 9, 2020 14:44
@barambani barambani merged commit aa70dac into laserdisc-io:master Aug 9, 2020
@barambani barambani deleted the parallel-bench-tune branch August 9, 2020 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sirocchj sirocchj sirocchj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@barambani @sirocchj