[JCR-3374] Size estimate for AbstractBundlePersistenceManager.MISSING is too low

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.3, 2.1.2, 2.2, 2.3, 2.4, 2.5
• Fix Version/s: 2.5.1
• Component/s: jackrabbit-core
• Assignee: Unassigned
• Reporter: Marcel Reutegger
• Created: 04/Jul/12 13:17
• Updated: 06/Aug/12 14:16
• Resolved: 05/Jul/12 12:54

Description

A heap dump from a Jackrabbit 2.2 based system recently showed that the ConcurrentCache of the bundle persistence managers used a lot more memory than configured. One of them used 10 times the configured bundle size.

It turned out that the majority of the cache entries referenced AbstractBundlePersistenceManager.MISSING. The size estimate of 16 bytes is too low and does not account for the overhead that comes with the internal LinkedHashMap of the ConcurrentCache.

I think a more sensible value is 128 bytes, which is the sum of:

• 32 bytes for ConcurrentCache.E
• 64 bytes for LinkedHashMap.Entry
• 32 bytes for NodeId

The unified cache, which also includes missing NodePropBundle entries was introduced with JCR-2699.