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



Reply via email to