> On Nov 25, 2015, at 8:53 AM, Jaroslav Bachorik <jaroslav.bacho...@oracle.com> > wrote: > > Updated webrevs: > > http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/top > http://cr.openjdk.java.net/~jbachorik/8043138/webrev.01/jdk >
This looks fine to me. Is there any test referencing the packages in jdk.jvmstat.rmi? I only found @modules with "jdk.jvmstat/sun.jvmstat.monitor” "jdk.jvmstat/sun.jvmstat.montior.event” It’d be good to double check and adjust the module name if any. Mandy