Review Request for 7024172 "Move platform MXBeans to java.lang.management"

Alan Bateman Alan.Bateman at oracle.com
Sun Mar 27 21:29:56 UTC 2011


Mandy Chung wrote:
>
>
> 7024172: Move BufferPoolMXBean and PlatformLoggingMXBean to 
> java.lang.management
>
> Webrev at:
>     http://cr.openjdk.java.net/~mchung/jdk7/7024172/webrev.00/
> javadoc at:
>     http://cr.openjdk.java.net/~mchung/jdk7/7024172/api
>
I've reviewed the changes, I'm sure Éamonn will too. Mostly look okay to 
me, with a couple of small comments.

In Managamenet.getPlatformMXBean (L647 and L723) it relies on 
PlatformComponent.getSingletonMXBean throwing IAE. It might be more 
obvious to use isSingleton instead and to throw the IAE if not.

ManagementFactoryHelper L148 - is this LoggingMXBean interface needed 
(just wondering if PlatformLoggingImpl could just instead implement 
PlatformLoggingMXBean and j.u.logging.LoggingMXBean).

PlatformLoggingMXBean.getLoggerNames reads "Returns the list of 
currently registered loggers" where as it is actually the list of the 
names of the registered loggers.

Minor inconsistency in PlatformLoggingMXBean in that two of the methods 
have spaces around the parameters, whereas the other two don't.

-Alan.
 






More information about the core-libs-dev mailing list