On Mon, 18 May 2026 18:59:39 GMT, Alan Bateman <[email protected]> wrote:

> I checked the API docs in the latest commit and I have more comments on the 
> API changes:

I've been looking at using AttachNotSupportedException for some of the problems 
that can happen along the way.
That can include an early check that finds an unusable revival cache dir in the 
Map env, but also bad core file, or wrong platform (for dirs for library 
lookup, failure is not fatal, but badly specified revival cache dir is fatal).

AttachNotSupportedException is already used for some "bad vmid" problems, so 
although "not supported" is not the phrase I'd choose, it can be the right 
Exception to throw for various problems.

When we get as far as execuring jcmd, IOException seems to be the way to say it 
went wrong.

Will check into all these points....

-------------

PR Comment: https://git.openjdk.org/jdk/pull/31011#issuecomment-4481250129

Reply via email to