RFR: 8027209: javax/management/monitor/ThreadPoolAccTest.java fails intermittently

[Erik Gahlin]

Will fix, not used to see @test before imports. Same problem with 6849945.

Erik

Yekaterina Kantserova skrev 2013-10-30 15:40:
> Not a reviewer, so just en observation. Seems like there are 2 jtreg 
> headers now:
>
>   24 /*
>   25  * @test
>   26  * @bug 6222826
>   27  * @summary Test that each thread in the thread pool runs
>   28  *          in the context of the monitor.start() caller.
>   29  * @author Luis-Miguel Alventosa
>   30  * @run clean ThreadPoolAccTest
>   31  * @run build ThreadPoolAccTest
>   32  * @run main ThreadPoolAccTest
>   33  */
>
> and
>
>
>   49 /*
>   50  * @test
>   51  *
>   52  * @run main/othervm ThreadPoolAccTest
>   53  */
>
> The original one does not use "other" in @run.
>
> Thanks,
> Katja
>
>
> On 10/30/2013 03:13 PM, Erik Gahlin wrote:
>> Could I have a review of this intermittently failing test.
>>
>> Instead of doing Thread.sleep(2000) and assume principal is 
>> visible/set the test now spins until principal is available. I also 
>> cleaned up code to avoid warnings.
>>
>> Thanks
>> Erik
>>
>> Webrev:
>> http://cr.openjdk.java.net/~egahlin/8027209_2/
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8027209
>>
>