Re: RFR(S) 8205652: serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorStatRateTest.java fails

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

Pushed. Thanks! Serguei On 7/17/18 17:54, JC Beyler wrote: Hi all, Here is the webrev: http://cr.openjdk.java.net/~jcbeyler/8205652/webrev.02/ Thanks for the reviews and future push! Jc On Tue, Jul 17, 2018 at 2:29 PM Alex Menkov <alexey.men...@oracle.com> wrote: +1 --alex On 07/16/2018 12:58, serguei.spit...@oracle.com wrote: > Hi Jc, > > It looks good to me. > > Thanks, > Serguei > > On 7/16/18 10:58, JC Beyler wrote: >> Hi all, >> >> Small RFR to update a HeapMonitor test that had two issues: a test was >> wrong and the test was not allocating enough to get to an expected >> sample count. Instead of allocating 10 times more and hit some OOM on >> the test framework, the webrev allocates in chunks and gets the number >> of samples. >> >> I ran this 10k times on my machine and it passed. Serguei ran mach5 >> testing with it and said it looked good. >> >> Bug associated is: JDK-8205652 >> <https://bugs.openjdk.java.net/browse/JDK-8205652> >> Webrev is here: >> http://cr.openjdk.java.net/~jcbeyler/8205652/webrev.01/ >> <http://cr.openjdk.java.net/%7Ejcbeyler/8205652/webrev.01/> >> >> Thanks, >> Jc > -- Thanks, Jc