[OAK-5935] AbstractSharedCachingDataStore#getRecordIfStored should use the underlying cache.get

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.1, 1.6.2, 1.8.0
• Component/s: blob
• Assignee: Amit Jain
• Reporter: Amit Jain
• Created: 15/Mar/17 09:47
• Updated: 09/Jun/17 12:53
• Resolved: 26/May/17 11:26

Description

The AbstractSharedCachingDataStore#getRecordIfStored can possibly use cache.get(dataIdentifier.toString()) instead of cache.getIfPresent(dataIdentifier.toString()) to load the binary in the cache from the backend,
The access pattern has to be evaluated but possibly we always do a getStream() after retrieving a record. The GC code already bypasses this with OAK-4573.

Issue Links

• is related to: OAK-6283 FileCache should ignore when file evicted with replacement
• relates to: OAK-5934 AbstractSharedCachingDataStore initializes a loader which closes input stream retrieved prematurely