RFR: 8293503: gc/metaspace/TestMetaspacePerfCounters.java#Epsilon-64 failed assertGreaterThanOrEqual: expected MMM >= NNN [v2]
Thomas Schatzl
tschatzl at openjdk.org
Wed Sep 14 13:59:39 UTC 2022
On Wed, 14 Sep 2022 13:21:06 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:
>> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>>
>> lkorinth suggestion
>
> test/hotspot/jtreg/gc/metaspace/TestMetaspacePerfCounters.java line 245:
>
>> 243: PerfCounterSnapshot snap2 = new PerfCounterSnapshot();
>> 244:
>> 245: final int MaxAttempts = 10;
>
> This should be used or removed as well, right?
`MaxAttempts` is used in the code. I prefer to have a named constant than a raw embedded value almost every time, also this time, but I can be convinced to inline it as well. Declaring it in the for-loop only clutters the loop body.
-------------
PR: https://git.openjdk.org/jdk/pull/10239
More information about the hotspot-gc-dev
mailing list