tree 64f6bc758c436c0798d580d3f4065ae02bf93c31
parent a60ed0c1d987874bd0f53fdc21f690fefa5d8492
author Googler <noreply@google.com> 1583167715 -0800
committer Copybara-Service <copybara-worker@google.com> 1583167774 -0800

When a fetch for an Object[] returns, cache the unwrapped contents.

By keeping the cached value as the wrapping ListenableFuture<Object[]>, we have the potential of making multiple fetches for the same fingerprint. This is illustrated by the test case - a fetch for a transitive member will only retain the ListenableFuture until it is resolved, then the cache entry becomes eligible for GC even though the underlying Object[] is still in memory.

I don't know how often this happens, but the change seems harmless and we can use the number of NestedSet reads to see if it makes any difference.

RELNOTES: None.
PiperOrigin-RevId: 298361983