On Tue, 7 Feb 2023 15:40:48 GMT, Darragh Clarke <[email protected]> wrote:
>> Currently there is a race condition that can allow for too many >> 'idleConnections' in `ServerImpl` >> >> This PR adds a lock to make sure only one connection can be marked Idle at a >> time as well as a test that consistently failed before the change but which >> now passes. > > Darragh Clarke has updated the pull request incrementally with one additional > commit since the last revision: > > addressed comments Thanks for those suggestions, I changed the lock to synchronized, update that print that was wrong in the test and changed the test to us URIBuilder ------------- PR: https://git.openjdk.org/jdk/pull/12413
