On Fri, 24 Nov 2023 06:54:27 GMT, Jaikiran Pai <[email protected]> wrote:
> Right now, the sole usage of the `monitoredHostServiceLoader` instance is
> within a `synchronized (monitoredHosts) {...}` block within a method. So it
> wouldn't require this `assert`.
Okay, I guess part of me wonders why this field is needed in the first place.
Why can't getMonitoredHost just create a ServiceLoader instead instead of
trying to share between threads?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16805#discussion_r1404261936
