RFR (XXS): 8194824: Add gc/stress/gclocker/TestGCLockerWithParallel.java to the ProblemList file

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR (XXS): 8194824: Add gc/stress/gclocker/TestGCLockerWithParallel.java to the ProblemList file

Thomas Schatzl
Hi all,

  can I get reviews for this small change the quarantines the
gc/stress/gclocker/TestGCLockerWithParallel.java test?

We can't get even modifications of it to reliably pass yet, maybe due
to JDK-8192647, but in the meantime we want to remove the noise from
the CI system.

CR:
https://bugs.openjdk.java.net/browse/JDK-8194824
Webrev:
http://cr.openjdk.java.net/~tschatzl/8194824/webrev/


Thanks,
  Thomas
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXS): 8194824: Add gc/stress/gclocker/TestGCLockerWithParallel.java to the ProblemList file

Kim Barrett
> On Jan 9, 2018, at 7:39 AM, Thomas Schatzl <[hidden email]> wrote:
>
> Hi all,
>
>  can I get reviews for this small change the quarantines the
> gc/stress/gclocker/TestGCLockerWithParallel.java test?
>
> We can't get even modifications of it to reliably pass yet, maybe due
> to JDK-8192647, but in the meantime we want to remove the noise from
> the CI system.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8194824
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8194824/webrev/
>
>
> Thanks,
>  Thomas

Looks good.

Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXS): 8194824: Add gc/stress/gclocker/TestGCLockerWithParallel.java to the ProblemList file

Erik Helin-2
In reply to this post by Thomas Schatzl
On 01/09/2018 01:39 PM, Thomas Schatzl wrote:

> Hi all,
>
>    can I get reviews for this small change the quarantines the
> gc/stress/gclocker/TestGCLockerWithParallel.java test?
>
> We can't get even modifications of it to reliably pass yet, maybe due
> to JDK-8192647, but in the meantime we want to remove the noise from
> the CI system.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8194824
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8194824/webrev/

Looks good, Reviewed.

Thanks,
Erik

> Thanks,
>    Thomas
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XXS): 8194824: Add gc/stress/gclocker/TestGCLockerWithParallel.java to the ProblemList file

Thomas Schatzl
Hi Kim, Erik,

On Wed, 2018-01-10 at 07:29 +0100, Erik Helin wrote:
> On 01/09/2018 01:39 PM, Thomas Schatzl wrote:
> > Hi all,
> >
> >    can I get reviews for this small change the quarantines the
> > gc/stress/gclocker/TestGCLockerWithParallel.java test?
> > [...]

  thanks for your reviews.

Thomas

> >