Any arguments against updating the AbstractEntityManagerImpl ctor to not include the (now) unused PersistenceContextType?