RFR: 8376052: Use AttachOperationFailedException rather than AttachNotSupportedException in findTargetProcessTmpDirectory() [v3]

Alan Bateman alanb at openjdk.org
Mon Jan 26 11:53:16 UTC 2026


On Sun, 25 Jan 2026 11:02:32 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:

>> [JDK-8373867](https://bugs.openjdk.org/browse/JDK-8373867) improves the logic to find tmp directory for using in attaching procedure. It might throw `IOException` or `AttachNotSupportedException` when it failed.
>> 
>> In review thread #28867, we reached conclusion we should use `AttachOperationFailedException` rather than `AttachNotSupportedException` because `AttachNotSupportedException` would be thrown when a compatible AttachProvider does not exist.
>
> Yasumasa Suenaga has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Update comments in findTargetProcessTmpDirectory()
>  - Throw AssertionError when URISyntaxException happens at findTargetProcessTmpDirectory()

Marked as reviewed by alanb (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/29355#pullrequestreview-3705692383


More information about the serviceability-dev mailing list