Looks good! Thanks, /Staffan
> On 15 apr 2015, at 16:46, Yekaterina Kantserova > <yekaterina.kantser...@oracle.com> wrote: > > Hi, > > Could I please have a review of this fix. > > bug: https://bugs.openjdk.java.net/browse/JDK-8077423 > webrev: http://cr.openjdk.java.net/~ykantser/8077423/webrev.00/ > > A couple of comments about changes in sun/tools/jstatd/JstatdTest.java. If > the suggested fix will be accepted there will be no need of > waitForJstatdRMI(), because jstad will either be started properly or will > terminate. > > A check has been added that there is no RMi Registry already running on > default port. Otherwise the test will fail fast. > > A sanity check for "-nr" option has been added as well. > > http://docs.oracle.com/javase/7/docs/technotes/tools/share/jstatd.html > "-nr > Do not attempt to create an internal RMI registry within the jstatd > process when an existing RMI registry is not found." > > > Thanks, > Katja