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