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

Alan Bateman alanb at openjdk.org
Thu Jan 22 07:30:04 UTC 2026


On Thu, 22 Jan 2026 06:40:06 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 one additional commit since the last revision:
> 
>   Fix typo

src/jdk.attach/linux/classes/sun/tools/attach/VirtualMachineImpl.java line 301:

> 299:             } catch (MonitorException | URISyntaxException e) {
> 300:                 // Other exceptions (happened at MonitoredHost) would be wrapped with AttachOperationFailedException
> 301:                 throw new AttachOperationFailedException("Unable to find target proces", e);

Would it be possible to paste in some examples of the cause?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29355#discussion_r2715649466


More information about the serviceability-dev mailing list