Hi Brent,
Thank you for reviewing.
Please check the new webrev:
http://cr.openjdk.java.net/~mli/8085192/webrev.01/, and comments inline
below.
On 2016/9/28 2:07, Brent Christian wrote:
Thanks for making some improvements to these intermittent RMI tests.
I agree with Roger that I don't think we want to add the id to the
@bug of every test.
Yes, I think it's reasonable too. Fixed.
Also, it looks like there's an indentation change in JavaVM.java:
53 public static final long POLLTIME_MS = 100L;
(I believe running webrev with '-b' will include whitespace changes.)
Fixed, thank you for the tip, new webrev is generated with "-b". :-)
Thank you
-Hamlin
Thanks,
-Brent
On 9/27/16 2:22 AM, Hamlin Li wrote:
Please review the fix for JDK-8085192. The fix checks whether it fails
to launch rmid due to "Port already in use" error, it will launch rmid
again and again(20 times at most) until no such issue.
bug:
https://bugs.openjdk.java.net/browse/JDK-8085192
webrev:
http://cr.openjdk.java.net/~mli/8085192/webrev.00/
Thank you
-Hamlin
