ROL-2101 Thread unsafe use of HashMap for cached mappings exist

Author: nuzayats

app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAUserManagerImpl.java

@@ -27,6 +27,7 @@
 import org.apache.roller.weblogger.business.UserManager;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.Date;
 import java.util.HashMap;
 import java.util.List;
@@ -49,7 +50,7 @@ public class JPAUserManagerImpl implements UserManager {
     private final JPAPersistenceStrategy strategy;
 
     // cached mapping of userNames -> userIds
-    private Map<String, String> userNameToIdMap = new HashMap<String, String>();
+    private Map<String, String> userNameToIdMap = Collections.synchronizedMap(new HashMap<String, String>());
 
 
     @com.google.inject.Inject

app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAWeblogEntryManagerImpl.java

@@ -68,7 +68,7 @@ public class JPAWeblogEntryManagerImpl implements WeblogEntryManager {
     private final JPAPersistenceStrategy strategy;
 
     // cached mapping of entryAnchors -> entryIds
-    private Map<String, String> entryAnchorToIdMap = new HashMap<String, String>();
+    private Map<String, String> entryAnchorToIdMap = Collections.synchronizedMap(new HashMap<String, String>());
 
     private static final Comparator<TagStat> TAG_STAT_NAME_COMPARATOR = new TagStatComparator();