RFR: JDK-8267371: Concurrent gtests take too long [v2]
Thomas Stuefe
stuefe at openjdk.java.net
Wed May 19 16:38:46 UTC 2021
On Wed, 19 May 2021 16:03:09 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
> Hi Thomas, you said `The patch reduces the timeout to 5 seconds, which reduces the time for one gtest by 30 seconds from 80 seconds down to 50 seconds.`
>
> Does that mean that the 30 second saving come from some of the tests being timed out? Why then bother at all with them?
>
> It does sound like they should move out of tier1...
Hi Gerard,
the test don't time out; they run a fixed amount of time. Sorry, "Timeout" was a bad phrasing here.
Moving them out of tier1 (or alternatively just getting rid of them) would be fine but as a separate RFE. The former would require either splitting gtests into several units, some tier1 worthy, some not; or to reimplement them as jtreg tests (as they had been before). Both is significantly more effort than I want to put into right now.
Thanks, Thomas
-------------
PR: https://git.openjdk.java.net/jdk/pull/4110
More information about the hotspot-dev
mailing list