Thanks for your help Daniel, Could I get a second review and I'll prepare an updated webrev :) Jc
On Wed, Jul 25, 2018 at 7:42 AM Daniel D. Daugherty < daniel.daughe...@oracle.com> 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. > > Dan > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8208059 > > As always, thanks for your help! > Jc > > > -- Thanks, Jc