Integrated: 8258956: Memory Leak in StringCoding on ThreadLocal resultCached StringCoding.Result
naoto at openjdk.java.net
Thu Jan 14 17:02:03 UTC 2021
On Wed, 13 Jan 2021 18:41:05 GMT, Naoto Sato <naoto at openjdk.org> wrote:
> Please review the fix to this issue. Unused thread local StringCoding.Result is now wrapped with SoftReference, which can be GC'ed as needed. I confirmed it worked as expected with a test case (https://github.com/openjdk/jdk/blob/8b4e1e1cf8f5d753ed901406f73d67b21557fddb/test/jdk/java/lang/StringCoding/ResultCachedGCTest.java), but decided not to include it in this PR. This is because I purposefully chose the size of the byte array and max heap size which is fragile, and could become noise in a regression test run.
This pull request has now been integrated.
Author: Naoto Sato <naoto at openjdk.org>
Stats: 18 lines in 1 file changed: 9 ins; 0 del; 9 mod
8258956: Memory Leak in StringCoding on ThreadLocal resultCached StringCoding.Result
Reviewed-by: rriggs, alanb
More information about the core-libs-dev