PersistentData classes aren't thread-safe
ForumPersistentData and BlogPersistentData aren't thread-safe but they're accessed by multiple threads. They hold references to non-thread-safe collections. Callers make multiple calls assuming that the state won't change between calls, potentially causing NPEs and other less obvious bugs.