Currently Role Strategy instance stores a HashMap for various role types. Every time this data is fetched, the logic has to retrieve an item from the container. It causes some performance degradation, especially in the case of proper concurrency locks.
Generally there is no need to have a HashMap for just several items. I propose to replace it by direct RoleMap fields