记一次线上问题 → 事务去哪了

开心一刻

　　小羊：哎呀，前面有奶喝

　　狗妈：这谁呀，走开

　　小羊：我就喝点，能怎么的嘛

　　狗妈：你喝就喝，咋还上头了呢？

　　小羊：真香！

　　狗妈：这羊犊子，真硬核！

问题背景

　　一天早上，楼主兴致勃勃的逛着园子的时候，右下角的 QQ 头像嘀嘀嘀的闪了起来，定睛一看，哎我去，肾要开始疼了，不是，头要开始疼了

　　客服 MM：太躺，有个客户充值成功后，赠送的积分没有到账

　　楼主：是不是客户等级不够，不满足资格 ？

　　客服 MM：客户等级是够的，他之前的积分都正常到账了

　　楼主：之前的积分都到账了 ？ 哪个客户，我去看看

　　客服 MM：客户名是：xxx，对应的单号是：xxx，找到原因了跟我说下

　　楼主：好的，找到原因了第一时间通知你

泰坦是楼主在公司内的花名，也是楼主的 LOL 本命英雄，慢慢的被传成太躺了，楼主也很无奈；
有小伙伴问楼主，你和客服 MM 什么关系，光看到头像闪动就肾疼了 ？
　　这个问题问得好，改天楼主给你加鸡腿，其实楼主和客服确实挺熟悉的，工作交流挺多的，但是仅限于同事关系！ 吾乃心系天下之人，岂能被儿女情长所困 ？ 只可惜客服 MM 已名花有主，不然就，嘿嘿嘿，你们懂的（是那姓吾的小子心系天下，楼主不姓吾！）

问题解决

　　积分赠送是最近新上的一个功能，上了也有一个多星期了，到目前为止，也就这个客户反馈了这个问题，另外这个客户之前的积分都是赠送到账了的，应该是触发了某些未考虑到的边界条件，产生了异常，导致积分未写入成功，照理来说，这应该是一个事务，要么都成功，要么都不成功呀

　　由于这个功能不是楼主开发的，出于快速解决问题的考虑，楼主就找到了对应的开发同事小李，跟他说明了下情况，让他去排查下什么原因

　　过了一会，小李找到了楼主，开始了他的排查分享

　　小李：太躺，我看了下日志，由于 xxx 情况未考虑到，导致加积分记录的时候抛异常了

　　楼主：xxx 情况确实比较特殊，一般很难考虑到，但是为什么存款成功了，积分却没加成功，你用了异步不 care 结果的处理 ？

　　小李：我是同步处理的，照理来说，应该要回滚的

　　楼主：那就奇了怪了，你把写入积分的方法给我下，我去看看代码

　　几分钟过后，楼主找到了小李，跟他说了下怎么改，并且让他把边界限制的处理也加上，走紧急流程升到了线上

　　问题解决后，小李又找到了楼主

　　小李：太躺啊，为什么之前事务未回滚，而按你说的那么改之后事务就会回滚了 ？

　　楼主：你去把你的椅子拿过来，我跟你好好讲讲！

问题复现

　　注意啊，这不是说升级了之后线上又出现了同样的问题，而是楼主为了让大家更好的了解这个问题，模拟下当时的场景

　　数据库版本 5.7.21 、存储引擎 InnoDB 、隔离级别 RR 、spring的传播机制 REQUIRED 、声明式事务 @Transactional 

　　完整代码：data-init，里面的 TransactionMissTest ，关键代码如下

/**
 * 存款
 * 引入积分之前的处理
 * @param loginName
 * @param amount
 * @return
 */
@Override
@Transactional(rollbackFor = Exception.class)
public TranMissCredit deposit(String loginName, BigDecimal amount) {
    TranMissCredit credit = creditMapper.getByLoginName(loginName);
    BigDecimal creditAfter = credit.getCredit().add(amount);

    TranMissCreditLog creditLog = new TranMissCreditLog(loginName, credit.getCredit(),
            amount, creditAfter, "充值: " + amount);
    credit.setCredit(creditAfter);

    creditMapper.update(credit);
    int count = creditLogMapper.insert(creditLog);

    return credit;
}

/**
 * 存款
 * 引入积分后的新增的方法
 * @param loginName
 * @param amount
 * @param integration
 * @return
 */
@Override
public TranMissCredit deposit(String loginName, BigDecimal amount, int integration) {
    TranMissCredit credit = deposit(loginName, amount);             // 复用之前的存款逻辑

    // 下面是新增的积分业务
    int integrationAfter = credit.getIntegration() + integration;
    TranMissIntegrationLog log = new TranMissIntegrationLog(loginName, credit.getIntegration(),
            integration, integrationAfter, "充值赠送积分: " + integration);
    credit.setIntegration(integrationAfter);

    creditMapper.update(credit);
    integrationLogMapper.insert(log);
    return credit;
}


// 调用的地方，相当于Controller
@Autowired
private IDepositService depositService;

@Test
public void deposit() {

    // 积分引入前的调用
    // TranMissCredit credit = depositService.deposit("zhangsan", new BigDecimal(100));

    // 积分引入后的调用
    TranMissCredit credit = depositService.deposit("zhangsan", new BigDecimal(100), 10);

}

View Code

　　看上去好像没毛病吧，楼主你不是蒙我了把 ？ 蒙没蒙你，咱们找焦点访谈

　　我们先看下初始状态，目前只有客户 zhangsan ，其额度 100 ，积分 10 

　　我们来手动造个异常，模拟边界条件的触发，修改新增的 deposit 方法

/**
 * 存款
 * 引入积分后的新增的方法
 * @param loginName
 * @param amount
 * @param integration
 * @return
 */
@Override
public TranMissCredit deposit(String loginName, BigDecimal amount, int integration) {
    TranMissCredit credit = deposit(loginName, amount);             // 复用之前的存款逻辑

    // 下面是新增的积分业务
    int integrationAfter = credit.getIntegration() + integration;
    TranMissIntegrationLog log = new TranMissIntegrationLog(loginName, credit.getIntegration(),
            integration, integrationAfter, "充值赠送积分: " + integration);
    credit.setIntegration(integrationAfter);

    // 模拟异常抛出
    if ("zhangsan".equals(loginName)) {
        throw new RuntimeException("触发边界条件");
    }
    creditMapper.update(credit);
    integrationLogMapper.insert(log);
    return credit;
}

View Code

　　我们来看看结果

　　哟嚯，额度加成功了，积分却没加成功，事务没生效！是不是有点懵 ？

问题分析

　　我们仔细观察下 deposit 方法，一个有 @Transactional 修饰，一个没有，就这一个差别；虽说只有这一个差别，但 Spring 却在幕后替我们完成了很多事情

　　Spring 事务原理

　　　　关于这个，我相信大家都能答上来一点，底层实现就是动态代理（你还不知道动态代理 ？那还不赶紧去看：设计模式之代理，手动实现动态代理，揭秘原理实现）

　　　　当 Spring 检查到 @Transactional ，会给目标对象创建一个代理对象，然后在代理对象中给目标对象中被 @Transactional 修饰的方法织入事务增强处理，类似这样

　　　　如果目标对象中没有被 @Transactional 修饰的方法，在代理类中是怎样的了 ？ 既然没有被 @Transactional ，说明不需要事务增强处理嘛，那就直调呗

　　　　回到我们的案例，代理对象与被代理对象之间的调用如下

　　　　可以看出来，目标对象新增的方法 TranMissCredit deposit(String loginName, BigDecimal amount, int integration) 在代理对象内是没有织入事务的，也就是默认的自动提交，那么异常抛出之前的数据库操作都是自动提交的，不会因后面的异常而回滚

　　　　其实不是事务丢失了，而是根本就不在一个事务中

　　再次校验

　　　　不只是 Spring 事务，很多的 AOP 也都一样，代码中直接操作的往往不是目标对象，而是目标对象的代理，通过代理对象来间接操作目标对象，而在代理对象中我们可以做一些前置或者后置的增强处理，不信 ？ 我们再次找焦点访谈

　　　　打个断点，看看就知道了

　　　　注入到 TransactionMissTest 的确实是代理对象

　　　　我们在 TranMissCredit deposit(String loginName, BigDecimal amount) 上打个断点，然后两种方式各调用一次，来看看调用链有什么不一样

　　　　以 depositService.deposit("zhangsan", new BigDecimal(100)); 方式调用时

　　　　此时调用链中有事务拦截器，有事务的调用链

　　　　以 depositService.deposit("zhangsan", new BigDecimal(100), 10) 方式调用时

　　　　此时调用链中没有事务拦截器，没有事务的调用链

　　　　是不是很明了了，so easy

总结

　　1、正常上线流程

　　　　线上问题 → 问题定位 → 问题复现 → 问题修复 → 转测试 → 测试通过升线上

　　　　而不是像文中说的那么轻描淡写

　　2、事务去哪了

　　　　Spring 事务的底层实现就是动态代理，是通过代理的方式对目标对象做前后的增强处理，前置开启事务、后置提交（回滚）事务；

　　　　增强处理在代理对象内，而不是在目标对象内，若目标对象的方法没有被 @Transactional 修饰，则在代理对象的代理方法内不会有关于事务的增强处理，而是直接调用目标对象的方法，那么后续的数据库操作就不是在一个事务中了

　　　　不是事务消失了，而是不在同一个事务了