【BUG】声明式事务失效导致日志记录失败

发生背景

公司里有一段代码,刚看到我是十分震惊的,我们知道在这种场景下事务会失效

  1. @Transactional 注解修饰的事务方法权限修饰符只能是 public,否则事务会失效
  2. 当前类中的方法不能调用事务方法,否则事务失效

然而公司内有一段向数据库中添加日志的代码,都存在导致事务失效的问题;带着事务是否会失效的疑问,自己亲自在本地实现了类似的代码。

问题复现

java 复制代码
@Transactional
@Service
public class UserServiceImpl {

    @Autowired
    private UserDao userDao;

    public void register(User user) {
	    //...
        save(user);
        //...
    }
    
	// 模拟添加日志到数据库
    @Transactional(propagation = Propagation.REQUIRES_NEW)
    private void save(User user) {
        userDao.save(user);
    }
}
java 复制代码
/**
 * <p>
 *  测试事务失效
 * @author zhl
 */
public class TransactionTest {
    @Test
    public void testTransaction(){
        ApplicationContext ctx = new AnnotationConfigApplicationContext(MyBatisAutoConfiguration.class);
        User user = new User();
        user.setName("zhangsan");
        user.setPassword("123456");
        UserServiceImpl userServiceImpl = (UserServiceImpl) ctx.getBean("userServiceImpl");
        userServiceImpl.register(user);
    }
}

被 @Transactional 修饰的save方法的事务失效,传播属性失效,由于当前类被 @Transactional 修饰,register 方法变成事务方法。并且通过控制台输出的日志,只有 register 方法开启了事务,save 方法事务确实失效了。

说明:权限修饰符不是 public 或 当前类中直接调用事务方法 都会导致事务失效。

验证一:事务方法权限修饰符不是 public,导致事务失效

  • 当前类实现 ApplicationContextAware ,从 Spring 容器中获取代理后的 UserServiceImpl.save 方法,让执行事务方法的途径正确(排除当前类直接调用事务方法导致的事务方法失效)
    1. 权限修饰符非 public,控制台只输出了一个 register 事务,原save方法事务失效;

    2. 权限修饰符为 public,控制台输出了两个事务,且在最外层通过除零运行时异常,发现只有最外层 register 方法事务异常回滚,内部 save 方法事务正常提交 ,说明 save 方法的传播属性正常生效。

验证二:本类方法直接调用本类的事务方法,导致事务失效

  • 将 save 方法的权限修饰符改为 public(排除权限修饰符非 public 导致的事务失效)
    1. 控制台只输出一个事务,说明原 save方法事务失效
    2. 当前类实现 ApplicationContextAware,从 Spring 容器中获取代理后的 UserServiceImpl.save 方法;发现控制台输出两个事务,且原 save 方法事务正常挂起 ,证明本类直接调用本类的事务方法,导致原事务方法失效。

结论

经过验证证明 save 方法事务失效。由于类上的 @Transactional 注解,为所有方法开启了事务,包括register方法,传播属性和隔离属性等都是默认值,如果在 save 方法外,register 方法中遇到异常,整个方法都会回滚,save 事务失效传播属性失效,同样随着外部方法回滚而回滚,save 方法插入数据库失败。

由于直接调用了 save 方法,save 方法没有被 Spring 加工代理,导致事务失效;

由于权限修饰符不是 public,导致在动态代理时,无法访问到事务方法,无法为该方法进行加工添加事务。

解决方法

从 Spring 容器获取 Bean,调用事务方法,并且修改事务方法的权限修饰符为 public

java 复制代码
@Transactional
@Service
public class UserServiceImpl implements ApplicationContextAware {

    private ApplicationContext applicationContext;

    @Autowired
    private UserDao userDao;

    public void register(User user) {
    	//...
        UserServiceImpl userServiceImpl = (UserServiceImpl) applicationContext.getBean("userServiceImpl");
        userServiceImpl.save(user);
        //...
    }
    
	//模拟日志记录
    @Transactional(propagation = Propagation.REQUIRES_NEW)
    public void save(User user) {
        userDao.save(user);
    }

    @Override
    public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
        this.applicationContext = applicationContext;
    }
}

建议在方法上使用 @Transactional 注解

相关推荐
nbsaas-boot37 分钟前
Java 正则表达式白皮书:语法详解、工程实践与常用表达式库
开发语言·python·mysql
岁忧38 分钟前
(LeetCode 面试经典 150 题 ) 11. 盛最多水的容器 (贪心+双指针)
java·c++·算法·leetcode·面试·go
chao_78941 分钟前
二分查找篇——搜索旋转排序数组【LeetCode】两次二分查找
开发语言·数据结构·python·算法·leetcode
CJi0NG42 分钟前
【自用】JavaSE--算法、正则表达式、异常
java
风无雨1 小时前
GO 启动 简单服务
开发语言·后端·golang
Hellyc1 小时前
用户查询优惠券之缓存击穿
java·redis·缓存
斯普信专业组1 小时前
Go语言包管理完全指南:从基础到最佳实践
开发语言·后端·golang
今天又在摸鱼2 小时前
Maven
java·maven
老马啸西风2 小时前
maven 发布到中央仓库常用脚本-02
java·maven
代码的余温2 小时前
MyBatis集成Logback日志全攻略
java·tomcat·mybatis·logback