记一次Spring @Transactional失效的排查过程

问题

压力测试时发现生成了相同的序号,根据日志分析发现select ... for update没有锁住某一行的数据,从而导致序号重复

排查

十分确定select ... for update的互斥机制,该行数据必定被锁,其他select ... for update/update/delete等SQL需要等待锁才能执行后续操作.
首先怀疑事务是否起作用,使用TransactionSynchronizationManager#getCurrentTransactionName打印当前的事务,结果竟然是null!!!没有事务自然不会互斥, 能解释为什么生成重复序号了.
剩下就是为什么明明加了@Transactional注解,但是没有开启事务了.
带着问题检查代码,有问题的代码如下图.这是一个责任链的设计模式,对于责任链的任一个节点,先调用doProcess执行子类自身的逻辑,如果成功则执行下一个succ节点,报错则执行下一个fail节点,直到责任链结束.
找到了声明事务的doProcess方法,他是在process里用this.doProcess调起的,我们都知道Spring的AOP事务要生效,必须使用 另一对象.方法 的形式才有效,这就是@Transactional失效的原因. 简而言之,不能在同一个类中使用没有@Transactional的方法调起有@Transactional的方法

public abstract class BaseController {
    private ScBaseController succNext = null;
    private ScBaseController failNext = null;

    public DataMessage process(DataMessage msg) throws ScBaseException {
        DataMessage respMsg = this.doProcess(msg);
        // 执行成功
        if (ExecuteResultEnum.SUCCESS.equals(respMsg.getExecuteResult())) {
            if (this.succNext != null) {
                return this.succNext.process(respMsg);
            } else {
                return respMsg;
            }
        }
        // 执行失败
        else {
            if (this.failNext != null) {
                return this.failNext.process(respMsg);
            } else {
                return respMsg;
            }
        }
    }
    
    @Transactional     // 这里的事务注解没有作用
    public abstract DataMessage doProcess(DataMessage msg);

教训

看过很多文章都说过不能在同一个类中使用没有@Transactional的方法调起有@Transactional的方法,这个是Spring动态AOP的机制决定的,必须紧记这个教训.至于为什么动态AOP会这样,后续如果写到这方面的文章再详细说明.
幸好在压力测试中发现这个问题,并没有造成线上故障.有些问题不是做几笔交易就能说明清楚,必须在高并发高压力的测试下才能验证系统的健壮性.