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

本文转自:https://www.cnblogs.com/youzhibing/p/12071391.html

问题背景

  一天早上,楼主兴致勃勃的逛着园子的时候,右下角的 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 修饰,则在代理对象的代理方法内不会有关于事务的增强处理,而是直接调用目标对象的方法,那么后续的数据库操作就不是在一个事务中了

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

posted @ 2020-01-02 14:30  keepsummer  阅读(136)  评论(0编辑  收藏  举报