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

相关推荐
都叫我大帅哥41 分钟前
深入浅出 Resilience4j:Java 微服务的“免疫系统”实战指南
java·spring cloud
2401_831501731 小时前
Linux之Zabbix分布式监控篇(二)
数据库·分布式·zabbix
秋林辉2 小时前
Jfinal+SQLite处理 sqlite数据库执行FIND_IN_SET报错
jvm·数据库·sqlite
Cao_Shixin攻城狮3 小时前
Flutter运行Android项目时显示java版本不兼容(Unsupported class file major version 65)的处理
android·java·flutter
古月-一个C++方向的小白5 小时前
C++11之lambda表达式与包装器
开发语言·c++
巴里巴气5 小时前
MongoDB复杂查询 聚合框架
数据库·mongodb
沐知全栈开发5 小时前
Eclipse 生成 jar 包
开发语言
Dcs5 小时前
还在用 Arrays.hashCode?Java 自己也能写出更快的版本!
java
杭州杭州杭州6 小时前
Python笔记
开发语言·笔记·python
tanyongxi667 小时前
C++ AVL树实现详解:平衡二叉搜索树的原理与代码实现
开发语言·c++