Hi Serguei, Here it is: http://cr.openjdk.java.net/~jcbeyler/8208059/webrev.01/
Let me know if you need anything else and thanks for your help! Jc On Wed, Jul 25, 2018 at 10:37 AM serguei.spit...@oracle.com < serguei.spit...@oracle.com> wrote: > Hi Jc, > > It looks good. > I'll push it after you send me a patch. > > > On 7/25/18 07:42, Daniel D. Daugherty wrote: > > On 7/25/18 10:20 AM, JC Beyler wrote: > > Hi all, > > There seems to be an intermittent failure with the > HeapMonitorInterpreterArrayTest. I believe it is due to the possibility of > a huge interval being chosen at the end of the test and GC arriving before > checking the samples. > > This fix should help alleviate it by reducing the interval to 100k and > also checking the garbage collected objects, could someone review it please? > > Webrev: http://cr.openjdk.java.net/~jcbeyler/8208059/webrev.00/ > > > > test/hotspot/jtreg/serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorTest.java > No comments. > > Thumbs up. We'll need this fix in both JDK11 and JDK12. > > > Okay, I've changed the 'Fix Version' to 11. > > Thanks, > Serguei > > > Dan > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8208059 > > As always, thanks for your help! > Jc > > > > -- Thanks, Jc
