On Fri, 28 Mar 2025 23:39:30 GMT, Chen Liang <li...@openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> docs > > There shouldn't be. > > 1. This just allocates a new promise for a single-threaded computation; the > promise just holds a field unless there is racy computeValue, which is rare. > Also, most of the access to ClassValue is going to find a present entry, so > the computation is rare and the impact may be even smaller. > 2. The Promise objects are always removed by the finally block in > `getFromHashMap`, so there should not be retained memory. Thanks for the clarifications, @liach 👍 ------------- PR Comment: https://git.openjdk.org/jdk/pull/24043#issuecomment-2763869787