TestNoNull is checking for backsliding on the use of NULL in the hotspot source files. The test only needs to be run once per change but it is defined as a regular tier1 test which then gets run for each os-arch combination, plus product and debug, and then again with a bunch of flags in tier3. This is just a waste of resources. At a minimum we should make the test flagless so that it doesn't run in tier3, but we can also restrict it to a single platform.
Before the change the test ran 27 times in tier1-3 in our CI. After it runs once.
The same applies to TestIncludesAreSorted.
Before the change the test ran 27 times in tier1-3 in our CI. After it runs once.
The same applies to TestIncludesAreSorted.
- links to
-
Commit(master)
openjdk/jdk/b3fab414
-
Review(master)
openjdk/jdk/28818