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

[GR-44559] Finish ThreadMXBean implementation for Native Image. #6101

Open
fniephaus opened this issue Mar 1, 2023 · 6 comments · May be fixed by #8430
Open

[GR-44559] Finish ThreadMXBean implementation for Native Image. #6101

fniephaus opened this issue Mar 1, 2023 · 6 comments · May be fixed by #8430

Comments

@fniephaus
Copy link
Member

SubstrateThreadMXBean implements com.sun.management.ThreadMXBean for Native Image and is not fully implemented yet, for example:

@Override
public long[] getAllThreadIds() {
throw VMError.unsupportedFeature(MSG);
}
@Override
public ThreadInfo getThreadInfo(long id) {
throw VMError.unsupportedFeature(MSG);
}
@Override
public ThreadInfo[] getThreadInfo(long[] ids) {
throw VMError.unsupportedFeature(MSG);
}
@Override
public ThreadInfo getThreadInfo(long id, int maxDepth) {
throw VMError.unsupportedFeature(MSG);
}
@Override
public ThreadInfo[] getThreadInfo(long[] ids, int maxDepth) {
throw VMError.unsupportedFeature(MSG);
}

To improve JMX support (#4732), some of these methods throwing VMError.unsupportedFeature() will be switched to return reasonable, but empty default values (e.g., new ThreadInfo[0]). While this allows some JMX tools to work, some key functionality of ThreadMXBean is still missing and needs to be implemented.

@oeresundsgruppen
Copy link

Will this be in v23 ?

@fniephaus
Copy link
Member Author

Will this be in v23 ?

@oeresundsgruppen unfortunately no and we still haven't found anyone who has the time to work on this.

@smthelusive
Copy link

@fniephaus I would like to help! Can I pick it up?

@fniephaus
Copy link
Member Author

I would like to help! Can I pick it up?

Sure, feel free to take a look and let me know if you need any help. :)

@smthelusive
Copy link

Hi @fniephaus! I have created this PR, please take a look when you have time 🙂

@smthelusive
Copy link

Hi @selhagani, I am no longer working on this, please see the comments in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In progress
3 participants