Integrated: 7903746: jtreg tests run extremely slow on some environments due to hostname lookups
Jaikiran Pai
jpai at openjdk.org
Thu Jul 11 03:41:06 UTC 2024
On Sun, 9 Jun 2024 06:31:56 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.
This pull request has now been integrated.
Changeset: 08d003a9
Author: Jaikiran Pai <jpai at openjdk.org>
URL: https://git.openjdk.org/jtreg/commit/08d003a9489875535c0b034b103616a607ea98ca
Stats: 76 lines in 5 files changed: 66 ins; 7 del; 3 mod
7903746: jtreg tests run extremely slow on some environments due to hostname lookups
Reviewed-by: jjg, msheppar
-------------
PR: https://git.openjdk.org/jtreg/pull/204
More information about the jtreg-dev
mailing list