[OPENJPA-1151] Cache the current-time timestamp in AbstractQueryCache.onTypesChanged()

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.3.0, 2.0.0
• Fix Version/s: 1.3.0, 2.0.0-M3
• Component/s: jdbc
• Patch Info: Patch Available
• Assignee: Donald Woods
• Reporter: Jody Grassel
• Created: 30/Jun/09 21:18
• Updated: 09/Mar/10 18:31
• Resolved: 15/Sep/09 18:46

Description

AbstractQueryCache.java contains the following block:

for (Object o: changedTypes)

{ String name = ((Class) o).getName(); if(!changedClasses.containsKey(name)) changedClasses.put(name, new Long(System.currentTimeMillis())); }

Suggesting caching the Long containing the entity type's last-modification timestamp outside of the for-loop to improve performance and reduce memory usage.