google / caliper

Micro-benchmarking library for Java
Apache License 2.0
816 stars 109 forks source link

Moe Sync #387

Closed ronshapiro closed 6 years ago

ronshapiro commented 6 years ago

This code has been reviewed and submitted internally. Feel free to discuss on the PR and we can submit follow-up changes as necessary.

Commits:

Add a private constructor to abstract Caliper modules. 392537e74a91c0f6b12fdaef2fd298a87c8e9586 -------

Reduce the max heap for benchmark model workers; it didn't need to be as high as it was, and that caused issues for Android devices. 84d1cf625ff01140bcfa2c7f1d2207b165966de5 -------

Increase the max heap for benchmark model workers, but not as high as it was before. I'd wrongly assumed that since we don't run any benchmark code, the amount of memory required should be consistent across all benchmarks. But code actually can get run when the benchmark class is initialized (static fields/initializers). I'll have to think about how to deal with that. 355942de1f4794ad01d957da8e4d022018b7ddab