[Bug] TargetConnectionManager#markConnectionInUse may not actually update cache metadata

[andrewazores]

The current implementation simply calls getIfPresent, which will return the current entry in the cache if any, but is unclear if it will actually update the associated metadata for write/access time as intended. This can or will cause long-lived connections to be forcibly closed and interrupt long-running operations.