[AMQ-4027] Add support for java 7 in AbstractJmxCommand

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.6.0
• Fix Version/s: 5.7.0
• Component/s: JMX
• Patch Info: Patch Available
• Assignee: Claus Ibsen
• Reporter: Didier Gutacker
• Created: 06/Sep/12 10:44
• Updated: 08/Sep/12 09:14
• Resolved: 08/Sep/12 09:14

Description

In the class org.apache.activemq.console.command.AbstractJmxCommand the method isSunJVM() checks wether the JVM is from Sun or not. This is done by comparing the system property "vendor" with the hardcoded string "Sun Microsystems Inc.". This prevents using the PID for connecting to the JVM via JMX locally when using java 7 because the vendor string is "Oracle Corporation".

A possible bugfix is to enhance the method like this:

public static boolean isSunJVM() {
return (getJVM().equals("Sun Microsystems Inc.") ||
getJVM().equals("Oracle Corporation"));
}