[OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out

2018-03-30 Thread Laurent Bourgès
Phil, Here is the ClipShapeTest change to fix timeout issue JBS: https://bugs.openjdk.java.net/browse/JDK-8200526 webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8200526.0/ I increased the timeout to 300s per test. I did a jtreg run on my machine (slowed down to 2Ghz) and it passed su

Re: [OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out

2018-04-03 Thread Sergey Bylokhov
Looks fine. On 30/03/2018 13:09, Laurent Bourgès wrote: Phil, Here is the ClipShapeTest change to fix timeout issue JBS: https://bugs.openjdk.java.net/browse/JDK-8200526 webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8200526.0/ I increased the timeout to 300s per test. I did a jtr

Re: [OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out

2018-04-04 Thread Laurent Bourgès
Do I need another approval ? Or it is enough to push this test only fix. Laurent Le mar. 3 avr. 2018 à 23:26, Sergey Bylokhov a écrit : > Looks fine. > > On 30/03/2018 13:09, Laurent Bourgès wrote: > > Phil, > > Here is the ClipShapeTest change to fix timeout issue > > > > JBS: https://bugs.op

Re: [OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out

2018-04-04 Thread Jayathirth D V
: Wednesday, April 04, 2018 1:05 PM To: Sergey Bylokhov Cc: 2d-dev Subject: Re: [OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out   Do I need another approval ?   Or it is enough to push this test only fix.   Laurent   Le mar. 3 avr. 2018 à 23:26, Sergey

Re: [OpenJDK 2D-Dev] [11] JDK-8200526 Test sun/java2d/marlin/ClipShapeTest.java times out

2018-04-04 Thread Phil Race
+1 from me. -Phil. > On Apr 4, 2018, at 12:35 AM, Laurent Bourgès > wrote: > > Do I need another approval ? > > Or it is enough to push this test only fix. > > Laurent > >> Le mar. 3 avr. 2018 à 23:26, Sergey Bylokhov a >> écrit : >> Looks fine. >> >> On 30/03/2018 13:09, Laurent Bourgès