Why not unify it in template?

Do you mean to move the getGlobalTransaction method to the template? If so, it should be a refactoring of the code and should not be done in this commit.
你的意思是把getGlobalTransaction 方法移动到模版里么 如果这样的话应该是对代码的一次重构,不应该在本次提交中完成

Do you mean to move the getGlobalTransaction method to the template? If so, it should be a refactoring of the code and should not be done in this commit. 你的意思是把getGlobalTransaction 方法移动到模版里么 如果这样的话应该是对代码的一次重构,不应该在本次提交中完成

我说的是在template中针对传入globaltransactional进行判空,如果是空的应该抛异常提示不允许为null
I'm talking about passing in globaltransactional in the template for null, if it's null it should throw an exception to indicate that it's not allowed to be null.

Why does DefaultSagaTransactionalTemplate.java do the processing, but no exception is thrown here?
为什么DefaultSagaTransactionalTemplate.java做了处理,而这里没有进行抛出异常?

private void cleanUp(GlobalTransaction tx) {
    if (tx == null) {
        throw new FrameworkException("Global transaction does not exist. Unable to proceed without a valid global transaction context.",
                FrameworkErrorCode.ObjectNotExists);
    }
    if (tx.getGlobalTransactionRole() == GlobalTransactionRole.Launcher) {
        TransactionHookManager.clear();
    }
}

Is this suitable?

I have repaired it in this way, please review.