我正在开发一个遗留代码库，在我们的自定义存储库中看到了一些令人不安的东西。 EntityManager 直接自动连接到回购中。例如：

@Repository
public class OperationRepository {

    // Yikes
    private final EntityManager entityManager;

    private QOperation operation = QOperation.operation;

    @Autowired
    public OperationRepositoryImpl(EntityManager entityManager) {
        this.entityManager = entityManager;
    }

    @Override
    public Optional<Operation> findOneByIdAndType(Long operationId, OperationType type) {
        JPAQuery<Operation> query = new JPAQuery<>(entityManager);
        QOperation qOperation = QOperation.operation;
        query.select(operation).from(operation)
                .where(
                        operation.type.eq(type),
                ...
    }
}

我的理解是，这是危险的，因为 EntityManager 不是线程安全的，在这里作为字段进行管理。我在我们的任何配置文件中都没有明确的定义来提供 EntityManager 所以我不确定spring在这种情况下做了什么。
我要开始修复使用 @PersistenceContext 但我想知道，这么长时间以来，这对生产有什么影响，可能会产生什么后果。
我们是否将连接池大小限制为存储库bean中单个EntityManager的数量？它还可能造成其他危险吗？还是我做的太多了？