On Wed, 27 Oct 2021 18:43:41 GMT, Larry-N <d...@openjdk.java.net> wrote:
>> This fix adds a cache of service provider classes to LoginContext (in >> particular, it's a cache of LoginModules classes). The approach helps to >> increase the performance of the LoginContext.login() method significantly, >> especially in a multi-threading environment. Service Loader is used for >> polling on Service Provider classes, without instantiating LoginModules >> object if Service Provider name doesn't match record in .config file. The >> set of service providers is cached for each Context Loader separately. >> This code passed successfully tier1 and tier2 tests on mac os. > > Larry-N has updated the pull request incrementally with one additional commit > since the last revision: > > Address review notices Have you removed the `isLoaded` flags? If they are still referenced in `Loader`, the two files will always be compiled before running the test. ------------- PR: https://git.openjdk.java.net/jdk/pull/5748