Re: RFR of JDK-8080550: java/rmi/server/useCustomRef/UseCustomRef.java failed with java.net.BindException intermittently

2016-12-02 Thread Chris Hegarty 

On 02/12/16 09:06, Hamlin Li wrote:

Would you please review the below patch?

bug: https://bugs.openjdk.java.net/browse/JDK-8080550
webrev: http://cr.openjdk.java.net/~mli/8080550/


Looks good Hamlin.

-Chris.



Thank you
-Hamlin


diff -r 99dd72e05060 test/java/rmi/server/useCustomRef/UseCustomRef.java
--- a/test/java/rmi/server/useCustomRef/UseCustomRef.javaFri Dec 02
00:47:00 2016 -0800
+++ b/test/java/rmi/server/useCustomRef/UseCustomRef.javaFri Dec 02
01:05:36 2016 -0800
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1998, 2012, Oracle and/or its affiliates. All rights
reserved.
+ * Copyright (c) 1998, 2016, Oracle and/or its affiliates. All rights
reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -86,7 +86,7 @@

 System.err.println("creating Registry...");

-registry = TestLibrary.createRegistryOnUnusedPort();
+registry = TestLibrary.createRegistryOnEphemeralPort();
 int port = TestLibrary.getRegistryPort(registry);
 /*
  * create object with custom ref and bind in registry

RFR of JDK-8080550: java/rmi/server/useCustomRef/UseCustomRef.java failed with java.net.BindException intermittently

2016-12-02 Thread Hamlin Li 

Would you please review the below patch?

bug: https://bugs.openjdk.java.net/browse/JDK-8080550
webrev: http://cr.openjdk.java.net/~mli/8080550/

Thank you
-Hamlin


diff -r 99dd72e05060 test/java/rmi/server/useCustomRef/UseCustomRef.java
--- a/test/java/rmi/server/useCustomRef/UseCustomRef.javaFri Dec 02 
00:47:00 2016 -0800
+++ b/test/java/rmi/server/useCustomRef/UseCustomRef.javaFri Dec 02 
01:05:36 2016 -0800

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1998, 2012, Oracle and/or its affiliates. All rights 
reserved.
+ * Copyright (c) 1998, 2016, Oracle and/or its affiliates. All rights 
reserved.

  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -86,7 +86,7 @@

 System.err.println("creating Registry...");

-registry = TestLibrary.createRegistryOnUnusedPort();
+registry = TestLibrary.createRegistryOnEphemeralPort();
 int port = TestLibrary.getRegistryPort(registry);
 /*
  * create object with custom ref and bind in registry