...
However, we're still not threadsafe, because multiple threads can be accessing the instance SoftHashMap "name" while one or more threads are changing that map. So, when we construct our SoftHashMap, we wrap it in a Collections.synchronizedMap(). JIRA issue for fix in uPortal 2.4.3 JIRA issue for fix in uPortal 2.5
Exceptions
While the IEntityNameFinder interface entitles us to throw Exception, we don't actually need to throw any checked Exceptions. So the local methods here are not declared to throw exceptions. Anyone accessing us as a PersonDirNameFinder, rather than as an IEntityNameFinder won't have to deal with these exceptions.
...