Code review. Given the class is only inherited by XmlMetaDataProvider and ProgrammaticMetaDataProvider which populate the map through their constructor, it just should be an immutable map, no CHM.