RFR: 8307653: Adjust delay time and gc log argument in TestAbortOnVMOperationTimeout
Guoxiong Li
gli at openjdk.org
Tue May 9 07:53:23 UTC 2023
On Tue, 9 May 2023 07:32:54 GMT, David Holmes <dholmes at openjdk.org> wrote:
> With only one value there is no point having the loop
Good caught. I just obey the previous code style and forget to refactor it. Will fix it.
> Why not just replace the shorter, failing, delay with the new longer one?
The longer one is useless actually. If the shorter one, `63423` previously, passed,
it means the gc time is less than `63423`. Then the longer one will always pass.
And jtreg would report `time out` in seraval minutes normally,
so the longer time `12388131` is useless.
So I want to remove the longer one and just remain the shorter one.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13878#discussion_r1188264968
More information about the hotspot-runtime-dev
mailing list