+1 from me. -Phil.
> On Apr 4, 2018, at 12:35 AM, Laurent Bourgès <bourges.laur...@gmail.com> > 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 <sergey.bylok...@oracle.com> 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.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 >> > successfully although the ClipShapeTest finished in 1200s in total: >> > >> > grep elapsed ClipShapeTest.jtr >> > elapsed=1209067 0\:20\:09.067 >> > elapsed time (seconds): 1.131 >> > elapsed time (seconds): 1.129 >> > elapsed time (seconds): 70.291 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 244.185 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 127.729 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 187.525 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 60.818 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 200.825 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 126.312 >> > elapsed time (seconds): 0.0 >> > elapsed time (seconds): 190.248 >> > >> > >> > Regards, >> > Laurent >> >> >> -- >> Best regards, Sergey.