Thank you. Leonid
> On Jun 4, 2018, at 5:09 PM, [email protected] wrote: > > Looks good. > I'll sponsor this change. > /Jesper > >> On 31 May 2018, at 19:59, Leonid Mesnik <[email protected]> wrote: >> >> Thank you for review. >> >> It becomes a part of hotspot_tier2_runtime now and executed in tier2. (Also >> in tier3 and higher tiers with other options.) >> >> Leonid >> >>> On May 31, 2018, at 10:47 AM, Vladimir Kozlov <[email protected]> >>> wrote: >>> >>> Looks fine. In what tier the test will be run? >>> >>> Thanks, >>> Vladimir >>> >>> On 5/31/18 10:11 AM, Leonid Mesnik wrote: >>>> Hi >>>> Please review following tiny fix which marks test >>>> serviceability/dcmd/compiler/CompilerQueueTest.java as intermittent and >>>> exclude it from tier1. >>>> The test intermittently fails (1-2 times in 1000 runs) because of >>>> https://bugs.openjdk.java.net/browse/JDK-8158374. >>>> Tier1 should be stable and don't contain any tests with known issues. It >>>> should always pass if there are no regression introduced by new fix. So >>>> this test should be excluded from tier1. >>>> Also fix marks this test with corresponding key like it is done for jdk, >>>> langtoools and nashorn tests. >>>> webrev: http://cr.openjdk.java.net/~lmesnik/8204103/ >>>> bug: https://bugs.openjdk.java.net/browse/JDK-8204103 >>>> I verified that tier1 runs and doesn't contain this test now. >>>> See more information about intermittent keyword in >>>> https://bugs.openjdk.java.net/browse/JDK-8075565 >>>> Leonid >> >
