rmannibucau commented on pull request #71: URL: https://github.com/apache/commons-logging/pull/71#issuecomment-1006061588
I dont see the weakref being re-set so it looks like the cache boost is lost and the behavior depends the jvm highly now so you can end up behaving as no cache is there at all. Guess the cache must move to a factory field (not static, no weak ref) and the weak ref surround the factory only to rely on a different ref tracking? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
