RFR: 8311639: Replace currentTimeMillis() with nanoTime() in jtreg/gc
Leo Korinth
lkorinth at openjdk.org
Fri Aug 18 09:23:27 UTC 2023
On Fri, 18 Aug 2023 09:06:15 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:
>> It is not to calculate elapsed time. It is to create a unique(ish) key for a hash table. The new code does not guarantee a unique value, but then neither did the old.
>
> I could possibly add some characters to the key if that is what bothers you. I could of course also use `currentTimeMillis` as it is nothing wrong with it. But I would prefer removing `currentTimeMillis` in all gc testing.
I will try to see if I can do it without using time at all.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15331#discussion_r1298220026
More information about the hotspot-gc-dev
mailing list