RFR: 7903746: jtreg tests run extremely slow on some environments due to hostname lookups [v3]

[Jaikiran Pai]

On Tue, 9 Jul 2024 08:09:13 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this change which proposes to address https://bugs.openjdk.org/browse/CODETOOLS-7903746?
>> 
>> As noted in that issue, jtreg determines the hotsname and populates that detail in the `TestResult`. Right now, it does it for every single test that's part of that test run. The hostname determination is expensive and on some system can be slow due to hostname resolutions. It's not necessary to do this lookup for every test and can be done just once for that test run. The commit in this PR does just that by looking it up once for that run and then making it available in the `TestResult` of each of the tests.
>> 
>> I have run this change against our JDK mainline in the CI for tier1 through tier6. This hasn't shown any regressions and I have manually verified that the hostname is correctly made available, like before, in the .jtr files of these tests.
>
> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Mark's suggestion - "Cached" instead of "Lazy"

Thank you Jon and Mark for the reviews. I'll wait for Jon to notice this latest update to this PR and unless there are any concerns with the change, I'll go ahead and integrate this PR later this week.

-------------

PR Comment: https://git.openjdk.org/jtreg/pull/204#issuecomment-2217330066