RFR: 8307653: Adjust delay time and gc log argument in TestAbortOnVMOperationTimeout [v3]
Guoxiong Li
gli at openjdk.org
Thu May 11 18:24:46 UTC 2023
On Thu, 11 May 2023 04:59:02 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove timeout option. Adjust comment. Level up the log to info.
>
> I am okay with these changes.
>
> I do worry that your slow dev board may expose a lot of internal timeout issues in tests - but tests should be using values modified by the timeoutFactor to compensate for slow systems where possible. Not sure how that would apply in the current case though.
@dholmes-ora @dcubed-ojdk Thanks for the review. Integrating.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/13878#issuecomment-1544472222
More information about the hotspot-runtime-dev
mailing list