RFR: 7903188: Log time spent waiting to acquire exclusive access lock [v2]

Jonathan Gibbons jjg at openjdk.org
Tue Jul 2 23:55:29 UTC 2024


On Thu, 20 Jun 2024 14:06:08 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java line 226:
>> 
>>> 224:      */
>>> 225:     @Deprecated
>>> 226:     public boolean needsExclusiveAccess(TestDescription td) throws TestSuite.Fault {
>> 
>> This already existing `throws TestSuite.Fault` is actually unnecessary because nothing in that logic throws that checked exception. However, I decided not to change this method signature since that can cause compatibility issue - I'm guessing this `com.sun.javatest.regtest.config.RegressionTestSuite` is a public API?
>> That's why I decided to introduce a new `isExclusiveAccessRequired` method which doesn't throw the checked exception.
>
> I am not aware of any consumer of `RegressionTestSuite` - is there, @jonathan-gibbons ?

`RegressionTestSuite` is a mostly-private implementation of a public interface `TestSuite`.
"mostly private" means that there are probably no other uses of the class. Maybe at one point I thought there might be.

I also suspect this might be a historical leftover from times past, when access to the underlying properties might have been more lazy.

At any rate, any clients of `RegressionTestSuite` would be "friends" and should be willing to tolerate change. There is no published commitment to the stability of these internal classes.

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

PR Review Comment: https://git.openjdk.org/jtreg/pull/208#discussion_r1663303210


More information about the jtreg-dev mailing list