I approve this change. @key intermittent also added in CVS. We also observe this test to fail, but only very rarely.
On Mon, Nov 9, 2015 at 7:44 PM, joe darcy <joe.da...@oracle.com> wrote: > Hi Amy, > > On 11/9/2015 7:20 PM, Amy Lu wrote: > >> java/util/concurrent/Phaser/Basic.java fails intermittently (JDK-8141031). >> >> This patch is to mark the test with keyword 'intermittent' and moved it >> to tier 2 until the problem is addressed. >> >> bug: https://bugs.openjdk.java.net/browse/JDK-8142370 >> webrev: http://cr.openjdk.java.net/~amlu/8142370/webrev.00/ >> >> Thanks, >> Amy >> >> > I will let Martin and other j.u.concurrent hackers comment whether or not > the test in question should be updated out-of-band from the syncs of JSR > 166alpha into the JDK. However, I do think putting this test in tier 2 > rather than tier 1 is appropriate until the intermittent problem is tracked > down. > > Thanks, > > -Joe >