RFR: 8320687: sun.jvmstat.monitor.MonitoredHost.getMonitoredHost() throws unexpected exceptions when invoked concurrently [v4]

[Jaikiran Pai]

On Fri, 24 Nov 2023 14:31:56 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> .map(mhs -> mhs.getMonitoredHost(hostId))

That was a reasoanable suggestion and I gave it a try, but `mhs.getMonitoredHost(hostId)` throws a checked exception `MonitorException` so using this `Stream` based approach then would need additional exception catching, wrapping and throwing, then catching this wrapped exception, checking the cause and rethrowing. That then defeats the purpose of using the `Stream` approach.

I see that you have approved the PR, so I think you are OK with the current non-stream approach. I'll now trigger the tier tests and if all comes back fine, I'll go ahead and integrate this in a day or two.

Kevin, does this latest form of the PR look OK to you too?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16805#discussion_r1404440378