eric6iese opened a new pull request, #2461: URL: https://github.com/apache/cxf/pull/2461
…instead of a WeakHashMap when timeToKeepState. This fixes a memory leak which occurs when the Thread is referenced within the ClientState to which the Thread points in the WeakHashMap. This scenario happens often, something with the new HttpClientConduit which internally might use a PipedInputStream referencing said thrad. I also added test to both check that the new implementation does not change the obversable behavior and a ruther blunt test which causes an OOME with the current implementation, but not with the new one. The tests are far from perfect because they use passive sleep statements instead of something like awaitility, but I did not want to write custom wait loops or integrate external libraries. -- 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: dev-unsubscr...@cxf.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
