On Fri, 15 Apr 2022 22:01:10 GMT, Paul Sandoz <psan...@openjdk.org> wrote:
>> Alan Bateman has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Refresh > > src/java.base/share/classes/jdk/internal/vm/ThreadContainers.java line 184: > >> 182: * with the Thread API. >> 183: */ >> 184: private static class RootContainer extends ThreadContainer { > > This implementation could be clearer if split out into two, and the value > assigned to `INSTANCE` is selected based on the system property. Something > to consider later perhaps. We've been undecided on whether to just track all threads. If we do that then there would be only one implementation. But yes, if we continue with two then they should be split out. ------------- PR: https://git.openjdk.java.net/jdk/pull/8166