ryomasaber / javamelody

Automatically exported from code.google.com/p/javamelody
0 stars 0 forks source link

Memory Histogram is not Supported under Mac OS X #80

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1.Click on the View Memory Hisogram.

What is the expected output? What do you see instead?
Memory Histogram should display.

What version of the product are you using? On what application server, JDK,
operating system?
1.23.0

Please provide any additional information below.
Under Mac OS X, "jmap -histo <pid>" displays memory histogram. Kindly update 
the variable inside VistualMachine.java as :

private static final boolean SUPPORTED = 
"1.6".compareTo(System.getProperty("java.version")) < 0
            && (System.getProperty("java.vendor").contains("Sun")
                    || System.getProperty("java.vendor").contains("Oracle") || JROCKIT || System.getProperty("java.vendor").contains("Apple"));

Original issue reported on code.google.com by saurabh1...@gmail.com on 7 Jan 2011 at 7:33

GoogleCodeExporter commented 8 years ago
Thanks. I have now included "Apple" as supported.
It is committed in trunk (revision 1621) and ready for the next release (1.25).

If should probably be tested in case of implementation details in the jdk, but 
I do not have mac os x for that.
Have you already tested a patched javamelody?

If not, you could do it with the new build which contains this change and which 
is available at:
http://javamelody.googlecode.com/files/javamelody-20110108.jar

Original comment by evernat@free.fr on 8 Jan 2011 at 9:10

GoogleCodeExporter commented 8 years ago
Yeah, though i m late in updating this issue. 

Its working good in 1.25.0 Build

Original comment by saurabh1...@gmail.com on 26 Jan 2011 at 12:11

GoogleCodeExporter commented 8 years ago

Original comment by evernat@free.fr on 1 Feb 2011 at 11:06