On Wed, 13 Jan 2021 18:41:05 GMT, Naoto Sato <na...@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. Changeset: aba3431c Author: Naoto Sato <na...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/aba3431c 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 ------------- PR: https://git.openjdk.java.net/jdk/pull/2064