RFR: 8065213 Specify and implement PlatformMBeanProvider for looking for all platform MBeans

Mandy Chung mandy.chung at oracle.com
Sat Jan 31 01:19:07 UTC 2015


On 1/30/2015 9:38 AM, shanliang wrote:
> Thanks for all your comments, here is the new version:
> http://cr.openjdk.java.net/~sjiang/JDK-8065213/01/

Thank you for the update.  Looks fine to me.  Minor comment:

PlatformMBeanProviderImpl.java
   line 43: better to rename it as "list" or other name.
   same as in DefaultPlatformMBeanProvider.java line 54

No need to generate new webrev.

Mandy


More information about the serviceability-dev mailing list