Re: RFR (XS) 8208059: [TESTBUG] HeapMonitorInterpreterArrayTest.java fails

[serguei.spit...@oracle.com]

Hi Jc, Pushed now. Thanks! Serguei On 7/25/18 10:54, JC Beyler wrote: 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