[SLING-4591] Performance: Consider using a faster empty map in ResourceMetadata#setParameterMap

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: API 2.9.0
• Fix Version/s: API 2.11.0
• Component/s: API
• Labels: perfomance
• Assignee: Tomek Rękawek
• Reporter: Joel Richard
• Created: 09/Apr/15 08:51
• Updated: 08/Feb/16 22:11
• Resolved: 03/Aug/15 10:06

Description

ResourceMetadata#setParameterMap is called several thousand times per request. In my case the passed parameterMap is always empty. Since it has to create a new empty LinkedHashMap for each call, 1-2% of the rendering time is spent in this method.

Therefore, I would suggest to either ignore empty maps or use Collections.emptyMap() instead. Both solutions should work if the caller does not make any undocumented assumptions for getParameterMap().