RFR: 7903746: jtreg tests run extremely slow on some environments due to hostname lookups [v3]
Jaikiran Pai
jpai at openjdk.org
Tue Jul 9 08:09:13 UTC 2024
> 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"
-------------
Changes:
- all: https://git.openjdk.org/jtreg/pull/204/files
- new: https://git.openjdk.org/jtreg/pull/204/files/f1943fc7..bee3b9a1
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jtreg&pr=204&range=02
- incr: https://webrevs.openjdk.org/?repo=jtreg&pr=204&range=01-02
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jtreg/pull/204.diff
Fetch: git fetch https://git.openjdk.org/jtreg.git pull/204/head:pull/204
PR: https://git.openjdk.org/jtreg/pull/204
More information about the jtreg-dev
mailing list