On 4/8/16 3:08 PM, Valerie Peng wrote:
Maybe you can just remove the comments on 103 and 104 for now.
I think this comment has good information. If okay, I will like to
retain it.
How did u come to the 600 timeout value? Just curious.
Nothing specific, it is hard to decide acceptable timeout value here.
Thanks,
Rajan
Rest looks fine.
Thanks,
Valerie
On 4/8/2016 2:27 PM, Rajan Halade wrote:
Please help with your review of following patch to
TestAlgParameterGenerator test. It allows test more time to run, not
run in othervm, and few more trivial updates.
Bug: https://bugs.openjdk.java.net/browse/JDK-8055351
Webrev: http://cr.openjdk.java.net/~rhalade/8055351/webrev.00/
Thanks,
Rajan
