【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 注解

相关推荐
Lu Yao_22 分钟前
Redis 持久化
数据库·redis·缓存
June`28 分钟前
Redis5安装与核心命令详解
数据库·redis·缓存
曾令胜3 小时前
excel导出使用arthas动态追踪方法调用耗时后性能优化的过程
spring·性能优化·excel
.格子衫.3 小时前
Spring Boot 原理篇
java·spring boot·后端
多云几多3 小时前
Yudao单体项目 springboot Admin安全验证开启
java·spring boot·spring·springbootadmin
安当加密3 小时前
达梦数据库TDE透明加密解决方案:构建高安全数据存储体系
网络·数据库·安全
摇滚侠5 小时前
Spring Boot 3零基础教程,Spring Intializer,笔记05
spring boot·笔记·spring
Jabes.yang6 小时前
Java求职面试实战:从Spring Boot到微服务架构的技术探讨
java·数据库·spring boot·微服务·面试·消息队列·互联网大厂
聪明的笨猪猪6 小时前
Java Redis “高可用 — 主从复制”面试清单(含超通俗生活案例与深度理解)
java·经验分享·笔记·面试
执尺量北斗6 小时前
[特殊字符] 基于 Qt + OpenGL 实现的入门级打砖块游戏
开发语言·qt·游戏