[LUCENE-10129] Add RamUsageEstimator shallowSizeOf(long[]) overload that just calls sizeOf(long[])?

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.0
• Lucene Fields: New
• Assignee: Uwe Schindler
• Reporter: Robert Muir
• Created: 28/Sep/21 13:46
• Updated: 28/Aug/22 16:26
• Resolved: 15/Oct/21 13:45

Description

See LUCENE-10128 for an example. The problem is there is only a sizeOf(long[]), so if the programmer uses shallowSizeOf instead of sizeOf then it falls back to shallowSizeOf(Object) which does a bunch of reflection.

This is pretty crazy because it can create performance traps. Should we just add a shallowSizeOf(long[]) that calls sizeOf(long[]), so that things are fast? (same for other primitive arrays). It would solve the problem easily I think.

Issue Links

• links to: GitHub Pull Request #367