Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#576 add SafepointExports #577

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

#576 add SafepointExports #577

wants to merge 3 commits into from

Conversation

mr-git
Copy link

@mr-git mr-git commented Oct 11, 2020

seems that the management API has been around since at least JDK 6 (http://srcrr.com/java/oracle/openjdk/6/reference/sun/management/HotspotRuntimeMBean-source.html)

@mr-git mr-git marked this pull request as ready for review October 11, 2020 18:44
@mr-git
Copy link
Author

mr-git commented Oct 11, 2020

@brian-brazil, please review - my local mvn is broken, but I checked the management API - it works at least on OpenJDK 14 and GraalVM 20.0.0 (java 1.8.0_242)

Copy link
Contributor

@brian-brazil brian-brazil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested it on the IBM JVM? It's not the hotspot ones I'm worried about.

@mr-git
Copy link
Author

mr-git commented Oct 20, 2020

Have you tested it on the IBM JVM? It's not the hotspot ones I'm worried about.

No, I haven't :(
Which IBM JVM would you recommend to try?

I found some links, that IBM is contributing to J9 and there is hotspot runtime mbean in place:
https://github.com/search?q=org%3AAdoptOpenJDK+getHotspotRuntimeMBean&type=code

@brian-brazil
Copy link
Contributor

Which IBM JVM would you recommend to try?

Whatever is easiest to get your hands on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants