On Mon, 4 Dec 2023 17:46:18 GMT, Tom Rodriguez <ne...@openjdk.org> wrote:
> This slightly increases the wait for reference processing to complete to
> accommodate long Xcomp compile times. Testing is underway.
test/jdk/java/lang/Object/FinalizationOption.java line 92:
> 90: System.gc();
> 91: try {
> 92: Thread.sleep(30L);
I think it would be better to scale these kinds of sleep time bounds by the
timeout factor used to run the jtreg test, which should already to scaled up
for known-to-be-slower testing scenarios.
There is a utility method to return this value in jdk/test/lib/Utils.java
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16956#discussion_r1414480356
