On Monday 25 September 2017 08:08 PM, Roger Riggs wrote:
Hi Vyom,

Looks fine to me too.

My only concern was adding an intensive test (used to be a stress test) to the normal test cycles.  I'd be more comfortable with it being in a separate/lower tier or as @manual.
thanks for review, i will make it "manual" before pushing as we already have similar test at lower tier.
Vyom

Roger



On 9/24/2017 1:15 PM, Chris Hegarty wrote:
On 15 Sep 2017, at 10:29, vyom tewari <vyom.tew...@oracle.com> wrote:

Hi,

Please review, small code change below.

webrev: http://cr.openjdk.java.net/~vtewari/8185072/webrev0.0/index.html
The changes look good to me Vyom. ( I only took a cursory look at the test changes )

-Chris.



Reply via email to