Patch: do not report percentiles that are just the maximum

Aleksey Shipilev shade at redhat.com
Wed Sep 26 14:33:33 UTC 2018


On 07/04/2018 05:32 PM, Erich Schubert wrote:
> The attached patch adds a threshold for the reported quantiles.
> 
> For example, for 10 measurements, it will report the 50% and 90% quantiles, but it will no longer
> report the 95%, 99%, 99.9%, 99.99%, 99.999% and 99.9999% quantiles as before, which were simply
> equal to the maximum, so completely redundant information & misleading.
> 
> The logic is straightforward. getMaxPercentile() returns the maximum percentile supported by the
> data, assumed to be the location of the last measurement on the empirical CDF. For anything larger
> than this, the getPercentile() function will simply return the maximum.

This makes sense. Let me see if that works.

Contribution wrinkle: OpenJDK patch acceptance rules need your OCA, as per
http://openjdk.java.net/contribute. Once you appear on this list:
https://www.oracle.com/technetwork/community/oca-486395.html -- we can accept the patch as is.

-Aleksey



More information about the jmh-dev mailing list