+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
